Skip to content

[VPU][TESTS][GNA] Fix dynamic models import on VPU#3427

Merged
iefode merged 6 commits intomasterfrom
unknown repository
Dec 8, 2020
Merged

[VPU][TESTS][GNA] Fix dynamic models import on VPU#3427
iefode merged 6 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Dec 1, 2020

Ticket - #-43979
Changes:

  • Fix for importing of dynamic models (remove internal shape inputs/outputs insertion to containers that are visible for users & fill descFromPlugin field)
  • Extract import/export tests from GNA to shared part in order to reuse it for other plugins

@ghost ghost added category: IE Tests OpenVINO Test: plugins and common category: GNA OpenVINO GNA plugin category: VPU labels Dec 1, 2020
@ghost ghost added this to the 2021.3 milestone Dec 1, 2020
@ghost ghost requested review from andrejsokolov, ggladilov and iefode December 1, 2020 08:13
@ghost ghost assigned ggladilov Dec 1, 2020
@ghost ghost self-requested a review December 1, 2020 08:13
@ghost ghost assigned iefode Dec 1, 2020
@ghost ghost self-requested a review as a code owner December 1, 2020 08:13
@ghost ghost assigned andrejsokolov and ghost Dec 1, 2020
@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 1, 2020

@andrejsokolov @gladilov-gleb could you please take a look?
@iefode could you please take a look at tests part?

@andrejsokolov andrejsokolov self-requested a review December 4, 2020 07:50
@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 4, 2020

@iefode could you please take a look once more?

Copy link
Copy Markdown
Contributor

@iefode iefode left a comment

Choose a reason for hiding this comment

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

In general, LGTM, beside one comment

@iefode
Copy link
Copy Markdown
Contributor

iefode commented Dec 8, 2020

Please, fix the CI and i will merge it

@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 8, 2020

@iefode CI is green.

@ghost ghost assigned iefode and unassigned ggladilov, iefode, andrejsokolov and ghost Dec 8, 2020
@iefode iefode merged commit e81201e into openvinotoolkit:master Dec 8, 2020
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Dec 11, 2020
)

* [VPU] Fix dynamic networks import

* [IE][GNA][TESTS] Move ImportExport tests from GNA to shared part

* [VPU][Tests] Add ExportImport test for dynamic network

* [VPU] Review fixes

* [VPU][Tests] Review and test fixes

* [VPU][Tests] Move TEST_P to shared part
vzinovie pushed a commit to vzinovie/openvino that referenced this pull request Dec 15, 2020
)

* [VPU] Fix dynamic networks import

* [IE][GNA][TESTS] Move ImportExport tests from GNA to shared part

* [VPU][Tests] Add ExportImport test for dynamic network

* [VPU] Review fixes

* [VPU][Tests] Review and test fixes

* [VPU][Tests] Move TEST_P to shared part
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Dec 16, 2020
)

* [VPU] Fix dynamic networks import

* [IE][GNA][TESTS] Move ImportExport tests from GNA to shared part

* [VPU][Tests] Add ExportImport test for dynamic network

* [VPU] Review fixes

* [VPU][Tests] Review and test fixes

* [VPU][Tests] Move TEST_P to shared part
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Jan 14, 2021
)

* [VPU] Fix dynamic networks import

* [IE][GNA][TESTS] Move ImportExport tests from GNA to shared part

* [VPU][Tests] Add ExportImport test for dynamic network

* [VPU] Review fixes

* [VPU][Tests] Review and test fixes

* [VPU][Tests] Move TEST_P to shared part
jiwaszki pushed a commit to akuporos/openvino that referenced this pull request Jan 15, 2021
)

* [VPU] Fix dynamic networks import

* [IE][GNA][TESTS] Move ImportExport tests from GNA to shared part

* [VPU][Tests] Add ExportImport test for dynamic network

* [VPU] Review fixes

* [VPU][Tests] Review and test fixes

* [VPU][Tests] Move TEST_P to shared part
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: GNA OpenVINO GNA plugin category: IE Tests OpenVINO Test: plugins and common

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants