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

Completely transition of MO to Interpolate-4 #970

Merged

Conversation

vgavrilo
Copy link
Contributor

@vgavrilo vgavrilo commented Jun 17, 2020

Completely transition of MO to Interpolate-4

Tickets: #38879, #44906

Code:

  • Comments
  • Code style (PEP8)
  • Transformation generates reshape-able IR: yes
  • Transformation preserves node names: yes

Validation:

  • Unit tests
  • Framework layer tests: layer tests for ONNX Resize-11
  • Transformation tests
  • e2e model test: N/A because there is no new model
  • MO IR Reader tests: N/A because there is no new model

Documentation:

  • Supported frameworks layers list: ONNX Resize (opset version >= 11)
  • Supported public models list: N/A because there is no new public model
  • New operations specification: operation Interpolate-4
  • Guide on how to convert the public model: N/A because there is no new public model
  • User guide update: N/A because there is no need to update common user guide

Other:

  • Sample/Demo application to infer the model: N/A since the model can be executed using existing classification sample

@vgavrilo vgavrilo changed the title WIP: Support of Interpolate-3 in MO WIP: Support of Interpolate-4 in MO Jul 9, 2020
vgavrilo added 14 commits July 16, 2020 15:14
@vgavrilo vgavrilo requested a review from ilyachur February 1, 2021 06:11
@vgavrilo
Copy link
Contributor Author

vgavrilo commented Feb 3, 2021

@ilyachur, please, see this PR (#970)

Copy link
Contributor

@GlebKazantaev GlebKazantaev left a comment

Choose a reason for hiding this comment

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

nGraph part looks ok.

@lazarevevgeny lazarevevgeny dismissed GlebKazantaev’s stale review February 5, 2021 16:16

Need to finally merge

@lazarevevgeny lazarevevgeny merged commit c1136cd into openvinotoolkit:master Feb 5, 2021
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) category: MO Model Optimizer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants