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

[GPU] Fixed friendly name of out transpose, improve Pad performance #8546

Merged
merged 2 commits into from
Nov 15, 2021
Merged

[GPU] Fixed friendly name of out transpose, improve Pad performance #8546

merged 2 commits into from
Nov 15, 2021

Conversation

vladimir-paramuzov
Copy link
Contributor

@vladimir-paramuzov vladimir-paramuzov commented Nov 11, 2021 •
edited
Loading

Details:

  • Fixed friendly name for transposes inserted by post-processing
  • Added border to fsv16 white list
  • Added fsv16 support for border primitive to avoid reorders

Tickets:

  • 69927
  • 70551
  • 70397

@vladimir-paramuzov vladimir-paramuzov added the category: GPU OpenVINO GPU plugin label Nov 11, 2021
@vladimir-paramuzov vladimir-paramuzov added this to the 2022.1 milestone Nov 11, 2021
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners November 11, 2021 14:37
@vladimir-paramuzov vladimir-paramuzov requested a review from a team November 11, 2021 14:37
@openvino-pushbot openvino-pushbot added the category: Core OpenVINO Core (aka ngraph) label Nov 11, 2021
lznamens
lznamens approved these changes Nov 12, 2021
View reviewed changes
Copy link
Contributor

@lznamens lznamens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please update corresponding JIRA tickets with info about perf improvements.

@vladimir-paramuzov vladimir-paramuzov merged commit 3b34f09 into openvinotoolkit:master Nov 15, 2021
openvino-dev-samples pushed a commit to openvino-dev-samples/openvino that referenced this pull request Nov 23, 2021
@vladimir-paramuzov @openvino-dev-samples
[GPU] Fixed friendly name of out transpose, improve Pad performance (o…
5ceca85 
…penvinotoolkit#8546)

* Fixed friendly names in post-processing nodes

* [GPU] Added fsv16 support for pad operation
openvino-dev-samples pushed a commit to openvino-dev-samples/openvino that referenced this pull request Nov 24, 2021
@vladimir-paramuzov @openvino-dev-samples
[GPU] Fixed friendly name of out transpose, improve Pad performance (o…
4bf52fd 
…penvinotoolkit#8546)

* Fixed friendly names in post-processing nodes

* [GPU] Added fsv16 support for pad operation
nosovmik added a commit to nosovmik/openvino that referenced this pull request Dec 20, 2021
@nosovmik
Leftover: Added tests for fix for PR openvinotoolkit#8546
c4bc3cf 
Added more preprocessing unit tests for fix of PR openvinotoolkit#8546
Testing: revert changes for the fix. Verify that new tests are failed
@nosovmik nosovmik mentioned this pull request Dec 20, 2021
Merged
ilya-lavrenov pushed a commit that referenced this pull request Dec 20, 2021
@nosovmik
Leftover: Added tests for fix for PR #8546 (#9313)
b7ffbe5 
Added more preprocessing unit tests for fix of PR #8546
Testing: revert changes for the fix. Verify that new tests are failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nosovmik nosovmik nosovmik approved these changes

@lznamens lznamens lznamens approved these changes

Assignees
No one assigned
Labels
category: Core OpenVINO Core (aka ngraph) category: GPU OpenVINO GPU plugin
Projects
None yet
Milestone
2022.1
Development

Successfully merging this pull request may close these issues.
4 participants
@vladimir-paramuzov @lznamens @nosovmik @openvino-pushbot