Skip to content

[Python API] Skip several tests if device is ARM#4646

Merged
akuporos merged 1 commit intoopenvinotoolkit:masterfrom
akuporos:fix/akup/skip-tests-arm
Mar 30, 2021
Merged

[Python API] Skip several tests if device is ARM#4646
akuporos merged 1 commit intoopenvinotoolkit:masterfrom
akuporos:fix/akup/skip-tests-arm

Conversation

@akuporos
Copy link
Copy Markdown
Contributor

@akuporos akuporos commented Mar 5, 2021 •
edited
Loading

Details:

  • skip tests where ngraph is imported
  • skip test test_get_perf_counts because layer_type on ARM is different from CPU

Tickets:

  • 46853

@akuporos akuporos requested review from a team and asuhov March 5, 2021 09:15
@openvino-pushbot openvino-pushbot added the category: Python API OpenVINO Python bindings label Mar 5, 2021
@akuporos akuporos added this to the 2021.4 milestone Mar 5, 2021
@ilya-lavrenov
Copy link
Copy Markdown
Contributor

ilya-lavrenov commented Mar 11, 2021

ARM plugin is going to be renamed to CPU as well. So, this skips will not work.
Secondly, I see that even some common simple tests have ben skipped like queryNetwork. I wonder is ARM does not support such simple layers

@akuporos
Copy link
Copy Markdown
Contributor Author

akuporos commented Mar 12, 2021

ARM plugin is going to be renamed to CPU as well. So, this skips will not work.
Secondly, I see that even some common simple tests have ben skipped like queryNetwork. I wonder is ARM does not support such simple layers

Thank you for your notice about the plugin name. The main problem with query_network test is that ngraph is used inside. As we discussed with @asuhov it isn't being built on ARM.

@akuporos akuporos force-pushed the fix/akup/skip-tests-arm branch from 6e7b41e to ca0aa2d Compare March 29, 2021 15:46
@akuporos akuporos force-pushed the fix/akup/skip-tests-arm branch from ca0aa2d to 171203f Compare March 29, 2021 15:52
@akuporos akuporos requested a review from nkogteva March 29, 2021 20:03
@akuporos akuporos merged commit a26aad1 into openvinotoolkit:master Mar 30, 2021
@akuporos akuporos deleted the fix/akup/skip-tests-arm branch March 30, 2021 12:17
luo-cheng2021 pushed a commit to luo-cheng2021/openvino that referenced this pull request Apr 7, 2021
@akuporos @luo-cheng2021
[Python API] Skip several tests if device is ARM (openvinotoolkit#4646)
30235d7
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Apr 23, 2021
@akuporos @mryzhov
[Python API] Skip several tests if device is ARM (openvinotoolkit#4646)
fe2e4cf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@nkogteva nkogteva nkogteva approved these changes

@asuhov asuhov asuhov approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ilya-lavrenov ilya-lavrenov

@nkogteva nkogteva

Labels

category: Python API OpenVINO Python bindings

Projects

None yet

Milestone

2021.4

Development

Successfully merging this pull request may close these issues.

5 participants

@akuporos @ilya-lavrenov @nkogteva @asuhov @openvino-pushbot