Skip to content

Removed backprop operations #1091

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

Merged

Conversation

ilyachur
Copy link
Contributor

@ilyachur ilyachur commented Jun 23, 2020

@GlebKazantaev @postrational Please take a look.

In this PR I removed back prop nGraph operations which are not exist in the opset1 and higher.

As a start of activity for removing v0 operations I removed v0 AvgPool operation from nGraph library and at the current moment it is used only in tests. Also I removed all unit tests which validate v0 AvgPool operation.

Also I removed downgrade pass to opset0 from nGraph library.

@ilyachur ilyachur force-pushed the feature/ichuraev/nGraph_clean branch 2 times, most recently from e2eda59 to 46caf33 Compare June 24, 2020 02:45
@ilyachur ilyachur added the category: Core OpenVINO Core (aka ngraph) label Jun 24, 2020
@ilyachur ilyachur added this to the 2021.1 milestone Jun 24, 2020
@ilyachur ilyachur force-pushed the feature/ichuraev/nGraph_clean branch 2 times, most recently from 8058522 to 28a429f Compare June 25, 2020 07:32
@ilyachur ilyachur marked this pull request as ready for review June 25, 2020 09:00
@ilyachur ilyachur requested review from a team, GlebKazantaev and postrational June 25, 2020 09:00
@ilyachur ilyachur force-pushed the feature/ichuraev/nGraph_clean branch 2 times, most recently from 340d1f6 to b0e52a4 Compare June 29, 2020 04:14
@ilyachur ilyachur force-pushed the feature/ichuraev/nGraph_clean branch from b0e52a4 to 938effc Compare June 29, 2020 06:20
@ilyachur ilyachur merged commit 182499c into openvinotoolkit:master Jun 29, 2020
@ilyachur ilyachur deleted the feature/ichuraev/nGraph_clean branch June 29, 2020 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants