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

Template reference tests #6456

Merged
merged 22 commits into from
Jul 12, 2021
Merged

Template reference tests #6456

merged 22 commits into from
Jul 12, 2021

Conversation

ilyachur
Copy link
Contributor

@ilyachur ilyachur commented Jun 30, 2021 •
edited
Loading

Details:

  • Moved nGraph Convert backend tests to template plugin
  • The idea is to use template plugin to check reference implementations and use SLT to compare IE plugins with reference implementations
  • This PR introduces the infrastructure for such migration and demonstrates the approach

Tickets:

  • NA

@ilyachur ilyachur added category: Core OpenVINO Core (aka ngraph) category: IE Tests OpenVINO Test: plugins and common labels Jun 30, 2021
@ilyachur ilyachur added this to the 2022.1 milestone Jun 30, 2021
@ilyachur ilyachur requested a review from a team as a code owner June 30, 2021 05:45
@ilyachur ilyachur requested review from a team June 30, 2021 05:45
@openvino-pushbot openvino-pushbot added the category: docs OpenVINO documentation label Jun 30, 2021
@ilyachur ilyachur force-pushed the template_reference_tests branch from af7bf11 to f89c127 Compare June 30, 2021 05:59
@ilyachur ilyachur mentioned this pull request Jul 2, 2021
Closed
@ilyachur
Copy link
Contributor Author

ilyachur commented Jul 5, 2021

@iefode @mikhail-treskin @ilya-lavrenov @jane-intel @GlebKazantaev @ggalieroc @pelszkow Please have a look

@ilyachur ilyachur requested review from a team July 7, 2021 12:42
ilyachur added 11 commits July 7, 2021 18:28
@bszmelcz bszmelcz mentioned this pull request Jul 12, 2021
Merged
@ilyachur ilyachur merged commit 811f4c5 into openvinotoolkit:master Jul 12, 2021
@ilyachur ilyachur deleted the template_reference_tests branch July 12, 2021 11:24
andrew-k-park pushed a commit to andrew-k-park/openvino that referenced this pull request Jul 14, 2021
@ilyachur
Template reference tests (openvinotoolkit#6456)
17207c5 
* Added convert tests

* Moved convert tests to template plugin

* Fixed build

* Fixed comments

* Fixed templateFuncTests

* Fixed review comments

* Added dynamic registration of template plugin

* Fixed functional tests

* Fixed behavior tests

* Added ticket

* Try to avoid fails
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
@ilyachur
Template reference tests (openvinotoolkit#6456)
a153f9e 
* Added convert tests

* Moved convert tests to template plugin

* Fixed build

* Fixed comments

* Fixed templateFuncTests

* Fixed review comments

* Added dynamic registration of template plugin

* Fixed functional tests

* Fixed behavior tests

* Added ticket

* Try to avoid fails
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Aug 30, 2021
@ilyachur @andrei-cv
Template reference tests (openvinotoolkit#6456)
06a1e96 
* Added convert tests

* Moved convert tests to template plugin

* Fixed build

* Fixed comments

* Fixed templateFuncTests

* Fixed review comments

* Added dynamic registration of template plugin

* Fixed functional tests

* Fixed behavior tests

* Added ticket

* Try to avoid fails
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
@ilyachur @akuporos
Template reference tests (openvinotoolkit#6456)
bb05671 
* Added convert tests

* Moved convert tests to template plugin

* Fixed build

* Fixed comments

* Fixed templateFuncTests

* Fixed review comments

* Added dynamic registration of template plugin

* Fixed functional tests

* Fixed behavior tests

* Added ticket

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

@pelszkow pelszkow pelszkow left review comments

@jdanieck jdanieck jdanieck approved these changes

@iefode iefode iefode approved these changes

@GlebKazantaev GlebKazantaev GlebKazantaev approved these changes

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

@jane-intel jane-intel Awaiting requested review from jane-intel

@mikhail-treskin mikhail-treskin Awaiting requested review from mikhail-treskin

@ggalieroc-zz ggalieroc-zz Awaiting requested review from ggalieroc-zz

Assignees
No one assigned
Labels
category: Core OpenVINO Core (aka ngraph) category: docs OpenVINO documentation category: IE Tests OpenVINO Test: plugins and common
Projects
None yet
Milestone
2022.1
Development

Successfully merging this pull request may close these issues.
7 participants
@ilyachur @ilya-lavrenov @GlebKazantaev @iefode @jdanieck @pelszkow @openvino-pushbot