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

Tests for dynamic preprocessing in SetBlob #5798

Merged
merged 6 commits into from
May 26, 2021
Merged

Tests for dynamic preprocessing in SetBlob #5798

merged 6 commits into from
May 26, 2021

Conversation

ilya-lavrenov
Copy link
Contributor

@ilya-lavrenov ilya-lavrenov commented May 25, 2021 •
edited
Loading

Details:

  • Current status of preprocessing:
    • input precision and layouts can be changes even after network compilation
    • outputs format (precision, layout) must match compiled one
  • CPU plugin is not able to convert input layout dynamically (TODO: create a bug)
  • MYRIAD plugin is able to convert even output layout dynamically (but we have common assert in SetBlob), while other plugins don't

Tickets:

@ilya-lavrenov ilya-lavrenov force-pushed the CVS-53713 branch 3 times, most recently from 6fcafd6 to a9bf863 Compare May 25, 2021 11:42
@azure-pipelines
Copy link

There was an error handling pipeline event 4a0afc24-4e86-4ccb-b6af-8c6b1952a7d2.

@ilya-lavrenov ilya-lavrenov changed the title Corrected tests to match CVS-53713 Tests for dynamic preprocessing in SetBlob May 25, 2021
@ilya-lavrenov ilya-lavrenov added this to the 2021.4 milestone May 26, 2021
@ilya-lavrenov ilya-lavrenov added category: inference OpenVINO Runtime library - Inference category: IE Tests OpenVINO Test: plugins and common labels May 26, 2021
@ilya-lavrenov ilya-lavrenov requested a review from ilyachur May 26, 2021 12:37
@ilya-lavrenov ilya-lavrenov marked this pull request as ready for review May 26, 2021 12:37
@ilya-lavrenov ilya-lavrenov requested a review from a team as a code owner May 26, 2021 12:37
@ilya-lavrenov ilya-lavrenov requested review from a team May 26, 2021 12:37
@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) May 26, 2021 14:18
@ilya-lavrenov ilya-lavrenov merged commit 26bfa6f into openvinotoolkit:master May 26, 2021
@ilya-lavrenov ilya-lavrenov deleted the CVS-53713 branch May 26, 2021 15:09
AlexeyLebedev1 pushed a commit to AlexeyLebedev1/openvino that referenced this pull request May 27, 2021
@ilya-lavrenov
Tests for dynamic preprocessing in SetBlob (openvinotoolkit#5798)
b81b059 
* Corrected tests to match CVS-53713

* Fixed tests configs

* Skip tests on GPU

* Commented condition inside main SetBlob because of MYRIAD

* Adopted tests
yekruglov pushed a commit to yekruglov/openvino that referenced this pull request Jun 7, 2021
@ilya-lavrenov @yekruglov
Tests for dynamic preprocessing in SetBlob (openvinotoolkit#5798)
eeac22f 
* Corrected tests to match CVS-53713

* Fixed tests configs

* Skip tests on GPU

* Commented condition inside main SetBlob because of MYRIAD

* Adopted tests
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
@ilya-lavrenov
Tests for dynamic preprocessing in SetBlob (openvinotoolkit#5798)
7b5f36d 
* Corrected tests to match CVS-53713

* Fixed tests configs

* Skip tests on GPU

* Commented condition inside main SetBlob because of MYRIAD

* Adopted tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilyachur ilyachur ilyachur approved these changes

Assignees
No one assigned
Labels
category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference
Projects
None yet
Milestone
2021.4
Development

Successfully merging this pull request may close these issues.
2 participants
@ilya-lavrenov @ilyachur