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

[REFERENCE] Fix Interpolate reference implementation #28471

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

[REFERENCE] Fix Interpolate reference implementation #28471

wants to merge 1 commit into from
+5 −5

Conversation

v-Golubev
Copy link
Contributor

@v-Golubev v-Golubev commented Jan 15, 2025 •
edited
Loading

Details:

  • This PR fixes Interpolate reference implementation: missed padding resizing is added. This logic was transferred from interpolate shape inference implemetation (for the details, please see interpolate::resize_padding implementation and usage).
  • The specification have no hard limitation that paddings size must be equal to input shape rank
  • LPT tests, that were affected by the issue, are reenabled

Tickets:

@v-Golubev v-Golubev requested review from a team as code owners January 15, 2025 21:37
@v-Golubev v-Golubev added this to the 2025.1 milestone Jan 15, 2025
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: GPU OpenVINO GPU plugin category: CPU OpenVINO CPU plugin labels Jan 15, 2025
@praasz praasz added this pull request to the merge queue Jan 16, 2025
@praasz praasz removed this pull request from the merge queue due to a manual request Jan 16, 2025
@praasz
Copy link
Contributor

praasz commented Jan 16, 2025

Added don't merge label as targeted for 25.1 release, should merged after CF

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@praasz praasz praasz approved these changes

@t-jankowski t-jankowski t-jankowski approved these changes

@mmikolajcz mmikolajcz mmikolajcz approved these changes

@mitruska mitruska mitruska approved these changes

Assignees
No one assigned
Labels
category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin category: GPU OpenVINO GPU plugin do not merge
Projects
None yet
Milestone
2025.1
Development

Successfully merging this pull request may close these issues.
5 participants
@v-Golubev @praasz @mitruska @mmikolajcz @t-jankowski