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

[OV20] NV12toRGB and NV12toBGR operations specification #7595

Merged
merged 3 commits into from
Sep 24, 2021
Merged

[OV20] NV12toRGB and NV12toBGR operations specification #7595

merged 3 commits into from
Sep 24, 2021

Conversation

nosovmik
Copy link
Contributor

Details:

  • NV12toRGB and NV12toBGR operations specification
  • It was decided to split this into 2 separate operations to be able to maintain their behavior separately. Also in futurethere may be needed to perform conversion from NV21, and this extension shall not affect users which use old operations (e.g. NV12toRGB-8 will be the same even if there is new NV21toRGB operation is introduced)

Tickets:

  • 62174

@nosovmik nosovmik requested a review from a team as a code owner September 22, 2021 10:50
@openvino-pushbot openvino-pushbot added the category: docs OpenVINO documentation label Sep 22, 2021
@ilya-lavrenov ilya-lavrenov added this to the 2022.1 milestone Sep 22, 2021
@nosovmik
Copy link
Contributor Author

@andrew-zaytsev can you please review this spec from your side?

tsavina
tsavina reviewed Sep 23, 2021
View reviewed changes
@nosovmik
Copy link
Contributor Author

@vinograd47 @dmitry-gorokhov can you please approve specification from CPU plugin side?

@jane-intel jane-intel merged commit ce9a229 into openvinotoolkit:master Sep 24, 2021
@nosovmik nosovmik mentioned this pull request Sep 29, 2021
Merged
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
@nosovmik @tsavina @akuporos
[OV20] NV12toRGB and NV12toBGR operations specification (openvinotool…
57b03b0 
…kit#7595)

* [OV20] NV12toRGB and NV12toBGR operations specification

* Changed supported type to "`uint8` or any supported floating-point type"

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

Co-authored-by: Tatiana Savina <[email protected]>
@nosovmik nosovmik mentioned this pull request Oct 28, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

@vinograd47 vinograd47 vinograd47 approved these changes

@ilyachur ilyachur ilyachur approved these changes

@vladimir-paramuzov vladimir-paramuzov vladimir-paramuzov approved these changes

@jane-intel jane-intel jane-intel approved these changes

@ArtemySkrebkov ArtemySkrebkov ArtemySkrebkov approved these changes

@tsavina tsavina tsavina approved these changes

@dmitry-gorokhov dmitry-gorokhov Awaiting requested review from dmitry-gorokhov

@andrew-zaytsev andrew-zaytsev Awaiting requested review from andrew-zaytsev

@artanokhov artanokhov Awaiting requested review from artanokhov

@taka-no-me taka-no-me Awaiting requested review from taka-no-me

@sshumihi sshumihi Awaiting requested review from sshumihi

Assignees
No one assigned
Labels
category: docs OpenVINO documentation
Projects
None yet
Milestone
2022.1
Development

Successfully merging this pull request may close these issues.
9 participants
@nosovmik @vinograd47 @ilya-lavrenov @ilyachur @vladimir-paramuzov @jane-intel @ArtemySkrebkov @tsavina @openvino-pushbot