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.

Sign up for GitHub

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

Jump to bottom

[transformations] Use descriptor API in ReverseShapeAndTypeInfer to update shapes, types #27594

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[transformations] Use descriptor API in ReverseShapeAndTypeInfer to update shapes, types #27594

wants to merge 1 commit into from
+90 −56

Conversation

praasz
Copy link
Contributor

@praasz praasz commented Nov 18, 2024

Details:

  • Remove friend of ReverseShapeAndTypeInfer from Tensor descriptor class
  • Setting the shapes in tensor descriptor directly not using the Tensor API can create risk e.g. PartialShape and Shape of Tensor descriptor will not be sync correctly.

Related PR:

Tickets:

@praasz praasz added this to the 2025.0 milestone Nov 18, 2024
@praasz praasz requested review from a team as code owners November 18, 2024 11:26
@praasz praasz requested review from itikhono and removed request for a team November 18, 2024 11:26
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: transformations OpenVINO Runtime library - Transformations category: CPP API OpenVINO CPP API bindings labels Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itikhono itikhono Awaiting requested review from itikhono itikhono is a code owner automatically assigned from openvinotoolkit/openvino-ie-transformations-maintainers

@t-jankowski t-jankowski Awaiting requested review from t-jankowski

@barnasm1 barnasm1 Awaiting requested review from barnasm1

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings category: transformations OpenVINO Runtime library - Transformations
Projects
None yet
Milestone
2025.0
Development

Successfully merging this pull request may close these issues.
1 participant
@praasz