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

[IE VPU] Execution graph via ngraph for VPU plugin #1572

Merged
merged 6 commits into from
Aug 20, 2020
Merged

[IE VPU] Execution graph via ngraph for VPU plugin #1572

merged 6 commits into from
Aug 20, 2020

Conversation

AntonDudchenko
Copy link
Contributor

#-33036

@AntonDudchenko AntonDudchenko requested a review from a team as a code owner July 31, 2020 08:38
@AntonDudchenko AntonDudchenko self-assigned this Jul 31, 2020
@AntonDudchenko AntonDudchenko marked this pull request as draft July 31, 2020 08:39
@AntonDudchenko AntonDudchenko requested a review from a user July 31, 2020 10:06
@AntonDudchenko AntonDudchenko marked this pull request as ready for review July 31, 2020 10:14
@AntonDudchenko AntonDudchenko requested a review from a team July 31, 2020 10:14
@ilya-lavrenov ilya-lavrenov added this to the 2021.1 milestone Aug 6, 2020
@openvino-pushbot openvino-pushbot added the category: IE Tests OpenVINO Test: plugins and common label Aug 11, 2020
@AntonDudchenko AntonDudchenko requested a review from KorA6 August 17, 2020 12:47
@ilya-lavrenov
Copy link
Contributor

@adudchen let's finalize by 2021.1

@ilya-lavrenov
Copy link
Contributor

@adudchen please, merge with latest master

@ilya-lavrenov
Copy link
Contributor

@adudchen please, have a look https://openvino-ci.intel.com/job/private-ci/job/ie/job/ie-tests-linux-ubuntu18-gna/12296/

Maxim-Doronin
Maxim-Doronin approved these changes Aug 19, 2020
View reviewed changes
Copy link
Contributor

@Maxim-Doronin Maxim-Doronin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some style issues. But in general LGTM

@Maxim-Doronin
Copy link
Contributor

@ilya-lavrenov, can we merge this PR right now?

@ilya-lavrenov
Copy link
Contributor

ilya-lavrenov commented Aug 20, 2020 •
edited
Loading

@Maxim-Doronin I think yes, since it passed review from VPU maintainers

@Maxim-Doronin Maxim-Doronin merged commit de8bfae into openvinotoolkit:master Aug 20, 2020
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Aug 26, 2020
@mryzhov
[IE][VPU]: Execution graph via ngraph for VPU plugin (openvinotoolkit…
4253211 
…#1572)

* Execution graph via ngraph for VPU plugin
* Enable ExecGraphTests
* Assert throw for multi device in tests
* Assert throw for gna device in tests
RomanZm pushed a commit to RomanZm/openvino that referenced this pull request Aug 28, 2020
[IE][VPU]: Execution graph via ngraph for VPU plugin (openvinotoolkit…
c587c33 
…#1572)

* Execution graph via ngraph for VPU plugin
* Enable ExecGraphTests
* Assert throw for multi device in tests
* Assert throw for gna device in tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilya-lavrenov ilya-lavrenov ilya-lavrenov left review comments

@Maxim-Doronin Maxim-Doronin Maxim-Doronin approved these changes

@KorA6 KorA6 KorA6 approved these changes

@DariaMityagina DariaMityagina Awaiting requested review from DariaMityagina

Assignees

@AntonDudchenko AntonDudchenko

Labels
category: IE Tests OpenVINO Test: plugins and common
Projects
None yet
Milestone
2021.1
Development

Successfully merging this pull request may close these issues.
6 participants
@AntonDudchenko @ilya-lavrenov @Maxim-Doronin @DariaMityagina @KorA6 @openvino-pushbot