Enable new tests for adjust_data_batch pass by DariaMityagina · Pull Request #1219 · openvinotoolkit/openvino

DariaMityagina · 2020-07-03T23:39:02Z

-32920
CI : 1346

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp

inference-engine/tests/unit/vpu/base/graph_transformer_tests.cpp

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp

inference-engine/tests/unit/vpu/base/graph_transformer_tests.cpp

andrejsokolov · 2020-07-10T13:31:32Z

@AndrewBakalinIntel, please continue review the request (I will be on vacation for the next 2 weeks)

DariaMityagina · 2020-07-15T07:29:18Z

@rvyunov @AndrewBakalinIntel
Could you please take a look?

inference-engine/tests/unit/vpu/base/graph_transformer_tests.cpp

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp

DariaMityagina · 2020-07-17T11:53:43Z

@rvyunov @AndrewBakalinIntel
Could you please take another look?

inference-engine/tests/unit/vpu/base/graph_transformer_tests.cpp

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp

DariaMityagina · 2020-07-22T21:23:19Z

@rvyunov @AndrewBakalinIntel
Could you please take another look?

DariaMityagina · 2020-07-24T15:02:04Z

@rvyunov do you have any comments?)

DariaMityagina · 2020-07-27T08:59:35Z

@Maxim-Doronin please review and merge if possible

inference-engine/tests/unit/vpu/base/graph_transformer_tests.hpp

DariaMityagina · 2020-07-28T10:51:54Z

IE-MDK fails on MyriadX now.
Going to look for a reason, either it started after the rebase or after my last commit.

DariaMityagina · 2020-07-29T08:43:53Z

IE-MDK fails on MyriadX now.
Going to look for a reason, either it started after the rebase or after my last commit.

It looks like the master branch has these failures as well (CI : 1322)

DariaMityagina · 2020-07-29T08:50:44Z

@Maxim-Doronin , could you take another look?

rvyunov · 2020-07-29T09:32:59Z

IE-MDK fails on MyriadX now.
Going to look for a reason, either it started after the rebase or after my last commit.

It looks like the master branch has these failures as well (CI : 1322)

It looks like infrastructure issue. I have same results. If you go deeper, than you can see all passed tests, but a few service steps will red.

…a constant

…ions.

…ess changes.

DariaMityagina · 2020-07-29T21:15:22Z

@Maxim-Doronin CI seems to be fine. Look again please

…it#1219) * New tests for adjust_data_batch pass

DariaMityagina requested a review from a team July 3, 2020 23:39

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from af7eb56 to 48e977f Compare July 8, 2020 08:01

ilya-lavrenov added category: IE Tests OpenVINO Test: plugins and common category: VPU labels Jul 8, 2020

ilya-lavrenov added this to the 2021.1 milestone Jul 8, 2020

DariaMityagina requested review from andrejsokolov and rvyunov July 8, 2020 10:46

rvyunov reviewed Jul 8, 2020

View reviewed changes

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp Show resolved Hide resolved

rvyunov reviewed Jul 8, 2020

View reviewed changes

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp Show resolved Hide resolved

andrejsokolov reviewed Jul 9, 2020

View reviewed changes

inference-engine/tests/unit/vpu/base/graph_transformer_tests.cpp Outdated Show resolved Hide resolved

andrejsokolov reviewed Jul 9, 2020

View reviewed changes

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp Outdated Show resolved Hide resolved

andrejsokolov reviewed Jul 9, 2020

View reviewed changes

inference-engine/tests/unit/vpu/middleend_tests/passes_tests/adjust_data_batch_tests.cpp Outdated Show resolved Hide resolved

andrejsokolov reviewed Jul 10, 2020

View reviewed changes

inference-engine/tests/unit/vpu/base/graph_transformer_tests.cpp Outdated Show resolved Hide resolved

andrejsokolov requested a review from a user July 10, 2020 13:28

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from 54d9e34 to fa6f818 Compare July 13, 2020 10:56

ghost reviewed Jul 15, 2020

View reviewed changes

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from d32f43c to 57dbe89 Compare July 17, 2020 09:21

ghost reviewed Jul 20, 2020

View reviewed changes

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from 377aa61 to 4e8b923 Compare July 22, 2020 08:03

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from 8bfb887 to 7431b4e Compare July 24, 2020 12:35

ghost approved these changes Jul 27, 2020

View reviewed changes

rvyunov approved these changes Jul 27, 2020

View reviewed changes

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from 7431b4e to 8a61827 Compare July 27, 2020 07:18

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from 8a61827 to 5ede266 Compare July 27, 2020 11:04

Maxim-Doronin reviewed Jul 27, 2020

View reviewed changes

inference-engine/tests/unit/vpu/base/graph_transformer_tests.hpp Outdated Show resolved Hide resolved

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch 2 times, most recently from 220a07f to d8fe4fb Compare July 28, 2020 08:27

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from d8fe4fb to f67140b Compare July 29, 2020 06:07

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from f67140b to ed8a132 Compare July 29, 2020 15:32

DariaMityagina added 16 commits July 29, 2020 22:20

[IE VPU] - new tests for adjust_data_batch pass

1860844

tests are disabled, because the fix is not yet in master

cae1634

code style failed for Mac. fix

3f7c88e

fixed according to the comments.

b3ca1ce

The variable is renamed, all its occurrences are changed

3a8eec5

corrections. not finished.

769f98e

reducing the amount of code + replacing the batchSize parameter with …

2fe4cc9

…a constant

code style fix

90aeb29

correcting tests and removing useless parameters

92ec7d7

correcting tests and trying to reuse existing functions

cca903f

trying to fix falures with constexpr. Making batchSize a constant.

225f802

correcting tests and trying to reuse existing functions

73d0a4e

[IE TESTS] Enable new tests for adjust_data_batch pass. Corrections.

73ead26

[IE TESTS] Enable new tests for adjust_data_batch pass. Small correct…

0f3a954

…ions.

[IE TESTS] Enable new tests for adjust_data_batch pass.

264566e

[IE TESTS] Enable new tests for adjust_data_batch pass. Removing usel…

6b7f690

…ess changes.

DariaMityagina force-pushed the icv/new_tests_for_adjust_data_batch branch from ebc496e to 6b7f690 Compare July 29, 2020 19:21

Maxim-Doronin approved these changes Jul 30, 2020

View reviewed changes

Maxim-Doronin merged commit e381062 into openvinotoolkit:master Jul 30, 2020

RomanZm pushed a commit to RomanZm/openvino that referenced this pull request Aug 28, 2020

[IE][VPU]: Enable new tests for adjust_data_batch pass (openvinotoolk…

bbe983b

…it#1219) * New tests for adjust_data_batch pass

Conversation

DariaMityagina commented Jul 3, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

andrejsokolov commented Jul 10, 2020

Uh oh!

DariaMityagina commented Jul 15, 2020

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

DariaMityagina commented Jul 17, 2020

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

DariaMityagina commented Jul 22, 2020

Uh oh!

DariaMityagina commented Jul 24, 2020

Uh oh!

DariaMityagina commented Jul 27, 2020

Uh oh!

Uh oh!

DariaMityagina commented Jul 28, 2020

Uh oh!

DariaMityagina commented Jul 29, 2020

Uh oh!

DariaMityagina commented Jul 29, 2020

Uh oh!

rvyunov commented Jul 29, 2020

Uh oh!

DariaMityagina commented Jul 29, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

DariaMityagina commented Jul 3, 2020 •

edited

Loading