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

Tensor names support in MO IR Reader #4194

Merged

Conversation

achetver
Copy link
Contributor

@achetver achetver commented Feb 5, 2021

Description: Add support for tensor names restoring and saving in MO IR Reader. Merge after related PR: #4089.

Ticket: 47403

Code:

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

Validation:

  • Unit tests: N/A - No new implemented functions;
  • Framework operation tests: N/A - No new enabled operations;
  • Transformation tests: N/A - No new implemented transformations;
  • e2e model test with an update of ./tests/e2e_oss/utils/reshape_utils.py: N/A - No affected changes;
  • Model Optimizer IR Reader check: Done manually.

Documentation:

  • Supported frameworks operations list: N/A - No new supported operations;
  • Supported public models list: N/A - Nothing to add;
  • New operations specification: N/A - No new supported operations;
  • Guide on how to convert the public model: N/A - Nothing to add;
  • User guide update: N/A - Nothing to update.

@achetver achetver added the category: MO Model Optimizer label Feb 5, 2021
@achetver achetver added this to the 2021.3 milestone Feb 5, 2021
@lazarevevgeny
Copy link
Contributor

@achetver , the [PR|https://github.com//pull/4242] has been merged. Please, rebase with the master branch.

@lazarevevgeny
Copy link
Contributor

@achetver , let's ask POT team to validate MO with this branch before merge.

@lazarevevgeny lazarevevgeny merged commit 2c4afd8 into openvinotoolkit:master Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: MO Model Optimizer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants