Skip to content

Completely transition of MO to Interpolate-4#970

Merged
lazarevevgeny merged 344 commits intoopenvinotoolkit:masterfrom
vgavrilo:vgavrilo/interpolate-3-mo-support
Feb 5, 2021
Merged

Completely transition of MO to Interpolate-4#970
lazarevevgeny merged 344 commits intoopenvinotoolkit:masterfrom
vgavrilo:vgavrilo/interpolate-3-mo-support

Conversation

@vgavrilo
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

vgavrilo commented Feb 3, 2021

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

Copy link
Copy Markdown
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