Skip to content

[VPU] Change StaticShapeNMS constructor to fix absent NMS-5 inputs#2880

Merged
Maxim-Doronin merged 1 commit intomasterfrom
unknown repository
Nov 5, 2020
Merged

[VPU] Change StaticShapeNMS constructor to fix absent NMS-5 inputs#2880
Maxim-Doronin merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Oct 28, 2020

Add new constructor to fix absent NMS-5 inputs that will be introduced after #2450 will be merged.

@ghost ghost changed the title [VPU] Fix NMS DTS [VPU] Change StaticShapeNMS constructor to fix absent NMS-5 inputs Nov 5, 2020
@ghost ghost marked this pull request as ready for review November 5, 2020 07:21
@ghost ghost self-requested a review as a code owner November 5, 2020 07:21
@ghost ghost added the category: VPU label Nov 5, 2020
@ghost ghost added this to the 2021.2 milestone Nov 5, 2020
@ghost ghost requested review from Maxim-Doronin and vgavrilo November 5, 2020 07:22
@ghost
Copy link
Copy Markdown
Author

ghost commented Nov 5, 2020

@Maxim-Doronin @vgavrilo could you please take a look?

@Maxim-Doronin Maxim-Doronin self-assigned this Nov 5, 2020
@ghost
Copy link
Copy Markdown
Author

ghost commented Nov 5, 2020

@Maxim-Doronin CI has passed. Could you please merge it? It is expected to be merged before #2450 otherwise the mentioned PR will introduce the regression in NMS models due to NMS-5 constructors changes.

@Maxim-Doronin Maxim-Doronin merged commit e758d2b into openvinotoolkit:master Nov 5, 2020
sdurawa pushed a commit to sdurawa/openvino that referenced this pull request Nov 6, 2020
Add a new constructor to fix absent NMS-5 inputs that will be introduced after openvinotoolkit#2450 will be merged.
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Dec 16, 2020
Add a new constructor to fix absent NMS-5 inputs that will be introduced after openvinotoolkit#2450 will be merged.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants