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.

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

Enable ngraph python tests in OpenVINO-ONNX CI #1603

Conversation

rblaczkowski
Copy link
Contributor

@rblaczkowski rblaczkowski commented Aug 3, 2020

Enable ngraph python tests in OpenVINO-ONNX CI:

  • set failing test cases as XFAIL with descriptions
  • unify ngraph and onnx tests
  • small refactor
  • add python ngraph tests to OpenVINO-ONNX CI

@rblaczkowski rblaczkowski requested a review from a team August 3, 2020 11:06
@rblaczkowski rblaczkowski marked this pull request as draft August 3, 2020 12:43
@rblaczkowski rblaczkowski changed the title Rblaczkowski/onnx ngraph python tests update Enable ngraph python tests in OpenVINO-ONNX CI Aug 4, 2020
@rblaczkowski rblaczkowski force-pushed the rblaczkowski/onnx-ngraph-python-tests-update branch from bb4aad8 to 6588a1f Compare August 4, 2020 12:33
@rblaczkowski rblaczkowski marked this pull request as ready for review August 4, 2020 14:33
@rblaczkowski rblaczkowski self-assigned this Aug 4, 2020
@rblaczkowski rblaczkowski requested a review from tomdol August 4, 2020 14:51
ngraph/python/tests/conftest.py Outdated Show resolved Hide resolved
ngraph/python/tox.ini Show resolved Hide resolved
 - add model zoo to onnx tests,
 - improvements of tests
@rblaczkowski rblaczkowski requested a review from a team August 6, 2020 07:19
@azhogov
Copy link

azhogov commented Aug 6, 2020

ONNX CI failed: unknown flag: --volume

@rblaczkowski
Copy link
Contributor Author

ONNX CI failed: unknown flag: --volume

Yes, I see, Im working for workaround

@ilya-lavrenov ilya-lavrenov added the category: CI OpenVINO public CI label Aug 6, 2020
@azhogov azhogov merged commit 054a7cd into openvinotoolkit:master Aug 7, 2020
RomanZm pushed a commit to RomanZm/openvino that referenced this pull request Aug 28, 2020
* Enable ngraph python tests

* Refactor and unify ngraph with onnx python tests

* Revert deprecated test cases

* Set ngraph and onnx python tests as a one test suite execution

* Change unstrict Xfails to strict ones

* Update after review:
 - add model zoo to onnx tests,
 - improvements of tests

* Revert mounting zoo models dir

Co-authored-by: Michał Karzyński <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CI OpenVINO public CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants