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 ScatterUpdate value infer #12595

Merged
merged 3 commits into from
Aug 22, 2022

Conversation

mvafin
Copy link
Contributor

@mvafin mvafin commented Aug 16, 2022

Details:

  • If ScatterUpdate appear in IR on shape route it will not calculate value and resulting shape will be determined as dynamic. This will result in failure of MO IR Reader when it will verify shapes after read.
  • This issue appeared only after ScatterElementsUpdate to ScatterUpdate transformation was enabled for offline step.

Tickets:

  • 89041

@mvafin mvafin requested a review from a team as a code owner August 16, 2022 19:30
@mvafin mvafin force-pushed the fix/ir_reader/scatter branch from 0ddf931 to d27878a Compare August 16, 2022 19:34
@akladiev
Copy link
Collaborator

@mvafin mvafin requested a review from rkazants August 18, 2022 18:46
@mvafin mvafin merged commit 56808c7 into openvinotoolkit:master Aug 22, 2022
@mvafin mvafin deleted the fix/ir_reader/scatter branch August 22, 2022 15:53
mvafin added a commit to mvafin/openvino that referenced this pull request Aug 23, 2022
* Add ScatterUpdate value infer

* Add additional test case to ScatterUpdate tests
andrei-kochin pushed a commit that referenced this pull request Aug 24, 2022
* Add ScatterUpdate value infer

* Add additional test case to ScatterUpdate tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants