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

[GPU] Shape infer for remaining ops #24440

Merged

Conversation

vladimir-paramuzov
Copy link
Contributor

@vladimir-paramuzov vladimir-paramuzov commented May 9, 2024

Details:

  • Added calc_output_layouts() impl for remaining operations, so since that patch all ops have new shape infer method, so we can start migration to the new path and removal of the legacy shape infer
  • Added RotateNMS handling in non_max_suppression_inst::calc_output_layouts
  • Pass max iterations count in loop shape infer
  • Added second out handling in pooling primitive
  • Added PSROIPooling and DeformablePSROIPooling handling in roi_pooling_inst::calc_output_layouts
  • Updated gpu tensor accessor to return empty tensor for non-existing dependency instead of exception to simplify usages

@vladimir-paramuzov vladimir-paramuzov added the category: GPU OpenVINO GPU plugin label May 9, 2024
@vladimir-paramuzov vladimir-paramuzov added this to the 2024.2 milestone May 9, 2024
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners May 9, 2024 11:13
@vladimir-paramuzov vladimir-paramuzov added this pull request to the merge queue May 14, 2024
github-merge-queue bot pushed a commit that referenced this pull request May 14, 2024
### Details:
- Added `calc_output_layouts()` impl for remaining operations, so since
that patch all ops have new shape infer method, so we can start
migration to the new path and removal of the legacy shape infer
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 14, 2024
@vladimir-paramuzov vladimir-paramuzov requested a review from a team as a code owner May 14, 2024 12:30
@github-actions github-actions bot added the category: Core OpenVINO Core (aka ngraph) label May 14, 2024
@vladimir-paramuzov vladimir-paramuzov added this pull request to the merge queue May 16, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 16, 2024
@vladimir-paramuzov vladimir-paramuzov added this pull request to the merge queue May 17, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 17, 2024
@vladimir-paramuzov vladimir-paramuzov added this pull request to the merge queue May 17, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 17, 2024
@p-durandin p-durandin added this pull request to the merge queue May 17, 2024
Merged via the queue into openvinotoolkit:master with commit 3f3c86d May 17, 2024
115 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph) category: GPU OpenVINO GPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants