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

Failed conversion from ONNX to IR #5742

Conversation

vgavrilo
Copy link
Contributor

Root cause analysis: According to https://phabricator.wikimedia.org/T213814, the method xml.etree.ElementTree.Element.getchildren() is deprecated from Python 3.2 and has been removed in Python 3.9. As a result, we have failure of conversion of some ONNX models, if the Python version is greater or equal than 3.9.

Solution: Replace element.getchildren() with list(element) in the file model-optimizer\mo\back\ie_ir_ver_2\emitter.py.

Ticket: 55705

Code:

  • Comments
  • Code style (PEP8)
  • Transformation generates reshape-able IR: N/A because there are no new transformations
  • Transformation preserves original framework node names: N/A because there are no new transformations

Validation:

  • Unit tests: N/A
  • Framework operation tests: N/A - No new enabled operations;
  • Transformation tests: N/A - No new implemented transformations;
  • Model Optimizer IR Reader check: N/A

Documentation:

  • Supported frameworks operations list: N/A - No new supported operations;
  • Supported public models list: N/A - Nothing to add;
  • User guide update: N/A - Nothing to update.

@vgavrilo vgavrilo requested review from a team, iimironov, sadolini, popovaan and lazarevevgeny and removed request for a team May 21, 2021 11:37
@openvino-pushbot openvino-pushbot added the category: MO Model Optimizer label May 21, 2021
@lazarevevgeny lazarevevgeny enabled auto-merge (squash) May 21, 2021 13:24
@lazarevevgeny lazarevevgeny merged commit 2bc69da into openvinotoolkit:master May 21, 2021
yekruglov pushed a commit to yekruglov/openvino that referenced this pull request Jun 7, 2021
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 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