Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add removing dangling results in MultiSubGraphOp transformation #15862

Merged

Conversation

mvafin
Copy link
Contributor

@mvafin mvafin commented Feb 21, 2023

Details:

  • Sometimes outputs in MultiSubGraphOp are unused, but they are not deleted, and they keep parts of bodies from deleting, which could result in inputs deleted and can help reduce number of operations in the model.

Tickets:

  • 102884

@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: transformations OpenVINO Runtime library - Transformations labels Feb 21, 2023
@mvafin mvafin requested a review from itikhono February 21, 2023 18:05
@github-actions github-actions bot added the category: PyTorch FE OpenVINO PyTorch Frontend label Feb 21, 2023
@mvafin mvafin marked this pull request as ready for review February 23, 2023 15:52
@mvafin mvafin requested review from a team as code owners February 23, 2023 15:52
@mvafin mvafin requested a review from eaidova February 23, 2023 15:52
@mvafin mvafin requested review from a team as code owners February 23, 2023 22:03
@github-actions github-actions bot added the category: IE Tests OpenVINO Test: plugins and common label Feb 23, 2023
@mvafin mvafin requested a review from itikhono February 27, 2023 06:40
@mvafin mvafin merged commit 2ced2ad into openvinotoolkit:master Feb 27, 2023
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Mar 21, 2023
…vinotoolkit#15862)

* Add removing dangling results in MultiSubGraphOp transformation

* Add recursive call for nested subgraphs

* Fix frontend build

* Add tests

* Add more tests and fix special port

* Add more tests, fix LSTM tests

* Preserve merged inputs

* Fix code style

* Fix paddle tests

* Fix special output
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference category: PyTorch FE OpenVINO PyTorch Frontend category: transformations OpenVINO Runtime library - Transformations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants