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

[WIP][GPU] Remove legacy shape infer and multi-out handling #25685

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[WIP][GPU] Remove legacy shape infer and multi-out handling #25685

wants to merge 4 commits into from
+1,690 −14,070

Conversation

vladimir-paramuzov
Copy link
Contributor

Details:

  • Completely removed legacy shape inference including calc_output_layout() method of primitives, allow_new_shape_infer property and all related calls
  • Removed legacy multi-output handling and migrate to new one
  • Cleanup api of several primitives to remove unused constructors and parameters
  • TODO:
    • LSTMCell/Sequence support to avoid decomposition to legacy subgraph
    • prepare_padding fixes
    • perf/accuracy checks and fixes

@vladimir-paramuzov vladimir-paramuzov added the category: GPU OpenVINO GPU plugin label Jul 23, 2024
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners July 23, 2024 07:33
@vladimir-paramuzov vladimir-paramuzov requested review from itikhono and removed request for a team July 23, 2024 07:33
@github-actions github-actions bot added category: CPU OpenVINO CPU plugin category: transformations OpenVINO Runtime library - Transformations labels Jul 23, 2024
@vladimir-paramuzov vladimir-paramuzov added no_stale Do not mark as stale WIP work in progress labels Jul 23, 2024
@github-actions github-actions bot removed category: CPU OpenVINO CPU plugin category: transformations OpenVINO Runtime library - Transformations labels Sep 4, 2024
@vladimir-paramuzov vladimir-paramuzov force-pushed the new_shape_infer branch 2 times, most recently from c27589f to d507c24 Compare September 11, 2024 07:16
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners November 26, 2024 07:19
@github-actions github-actions bot added category: IE Tests OpenVINO Test: plugins and common category: build OpenVINO cmake script / infra labels Nov 26, 2024
@github-actions github-actions bot removed category: IE Tests OpenVINO Test: plugins and common category: build OpenVINO cmake script / infra labels Dec 5, 2024
@vladimir-paramuzov
[GPU] Global migration to new shape infer and multi-out approach
ad02127 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
cleanup
38bec31 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
fixes
64773c6 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
fixes
779eda8 
Signed-off-by: Vladimir Paramuzov <[email protected]>
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 was automatically assigned from openvinotoolkit/openvino-ie-transformations-maintainers

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

Assignees
No one assigned
Labels
category: GPU OpenVINO GPU plugin no_stale Do not mark as stale WIP work in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@vladimir-paramuzov