Skip to content

[TRANSFORMATIONS] Fix implicit conversion of ov::Node with multiple outputs to ov::Output#29659

Merged
CuriousPanCake merged 2 commits intoopenvinotoolkit:masterfrom
CuriousPanCake:CVS-164465
Mar 25, 2025
Merged

[TRANSFORMATIONS] Fix implicit conversion of ov::Node with multiple outputs to ov::Output#29659
CuriousPanCake merged 2 commits intoopenvinotoolkit:masterfrom
CuriousPanCake:CVS-164465

Conversation

@CuriousPanCake
Copy link
Contributor

Details:

In the GeluFusionWithErfTwo transformation "input" may appear to be a VariadicSplit node with multiple outputs. Using the default output of the node is incorrect and leads to an exception if used as an input to another node because of an implicit conversion to ov::Output. Use the output of the VariadicSplit node explicitly to avoid the exception.

Tickets:

Signed-off-by: Andrii Staikov andrii.staikov@intel.com

@CuriousPanCake CuriousPanCake requested a review from a team as a code owner March 24, 2025 16:09
@CuriousPanCake CuriousPanCake requested review from itikhono and removed request for a team March 24, 2025 16:09
@github-actions github-actions bot added the category: transformations OpenVINO Runtime library - Transformations label Mar 24, 2025
@CuriousPanCake CuriousPanCake self-assigned this Mar 24, 2025
@CuriousPanCake CuriousPanCake added this pull request to the merge queue Mar 25, 2025
Merged via the queue into openvinotoolkit:master with commit 68845cd Mar 25, 2025
185 checks passed
@CuriousPanCake CuriousPanCake deleted the CVS-164465 branch March 25, 2025 14:44
timxu826 pushed a commit to timxu826/openvino that referenced this pull request Apr 7, 2025
…utputs to ov::Output (openvinotoolkit#29659)

### Details:
In the GeluFusionWithErfTwo transformation "input" may appear to be a
VariadicSplit node with multiple outputs. Using the default output of
the node is incorrect and leads to an exception if used as an input to
another node because of an implicit conversion to ov::Output. Use the
output of the VariadicSplit node explicitly to avoid the exception.

### Tickets:
- [CVS-164465](https://jira.devtools.intel.com/browse/CVS-164465)

Signed-off-by: Andrii Staikov <andrii.staikov@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: transformations OpenVINO Runtime library - Transformations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants