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

[VPU] Fix loading previous NMS versions #3696

Merged
merged 2 commits into from Dec 24, 2020
Merged

[VPU] Fix loading previous NMS versions #3696

merged 2 commits into from Dec 24, 2020

Conversation

ghost
Copy link

@ghost ghost commented Dec 22, 2020

Ticket - #-44281
Description: currently IRs with NMS version < 5 doesn't work because convertion of previous NMS versions to NMS-5 happens after DTS (in opset1 to legacy conversion), while for Myriad Plugin it's necessary to do so at the very beginning of ngraph convertion pipeline.

@ghost ghost added the category: VPU label Dec 22, 2020
@ghost ghost added this to the 2021.3 milestone Dec 22, 2020
@ghost ghost requested review from ggladilov and Maxim-Doronin December 22, 2020 10:05
@ghost ghost assigned ggladilov Dec 22, 2020
@ghost ghost self-requested a review December 22, 2020 10:05
@ghost ghost assigned Maxim-Doronin Dec 22, 2020
@ghost ghost self-requested a review as a code owner December 22, 2020 10:05
@ghost ghost self-assigned this Dec 22, 2020
@ghost
Copy link
Author

ghost commented Dec 22, 2020

@gladilov-gleb @Maxim-Doronin could you please take a look?

@openvino-pushbot openvino-pushbot added the category: IE Tests OpenVINO Test: plugins and common label Dec 22, 2020
@ghost
Copy link
Author

ghost commented Dec 24, 2020

@Maxim-Doronin could you please merge?

@ghost ghost assigned Maxim-Doronin and unassigned ggladilov, Maxim-Doronin and ghost Dec 24, 2020
@Maxim-Doronin Maxim-Doronin merged commit 29d82b7 into openvinotoolkit:master Dec 24, 2020
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Jan 14, 2021
* Description: currently IRs with NMS version < 5 don't work because conversion of previous NMS versions to NMS-5 happens after DTS (in opset1 to legacy conversion), while for Myriad Plugin it's necessary to do so at the very beginning of ngraph conversion pipeline.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: IE Tests OpenVINO Test: plugins and common
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants