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

Add mxnet extractors #1667

Conversation

iimironov
Copy link
Contributor

Description: Add mxnet asing acosh atanh ops
JIRA: 30101, 30093, 30085

Code:

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

Validation:

  • Unit tests: Not added, we use simple eltwise shape inference and value propagation functions
  • Framework operation tests: 36623
  • Transformation tests: No new transformation added
  • e2e model test with an update of ./tests/e2e_oss/utils/reshape_utils.py: No new supported models
  • Model Optimizer IR Reader check: Done

Documentation:

  • Supported frameworks operations list: Done, operation was supported via transformation, now it supported directly
  • Supported public models list: No new supported models
  • New operations specification: Done
  • Guide on how to convert the public model: No new supported models
  • User guide update: Nothing to update

@iimironov iimironov requested review from lazarevevgeny and a team August 6, 2020 13:04
@iimironov iimironov requested a review from a team as a code owner August 6, 2020 13:28
@iimironov iimironov force-pushed the imironov/CVS-30101-30093-30085-extend-python-mxnet-asinh-acosh-atanh-ops branch from 0f2192b to ebfa43f Compare August 7, 2020 08:15
@lazarevevgeny lazarevevgeny merged commit c8d7463 into openvinotoolkit:master Aug 7, 2020
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Aug 26, 2020
* Add mxnet extractors for hyperbolic functions
RomanZm pushed a commit to RomanZm/openvino that referenced this pull request Aug 28, 2020
* Add mxnet extractors for hyperbolic functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants