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

Transpose FQ optimization #5763

Conversation

jane-intel
Copy link
Contributor

@jane-intel jane-intel commented May 24, 2021

Solution: Transformation that pulls Transpose up through FQ was fixed to do nothing in case FQ has multiple consumers.
Fixed transformation order for optimization of the FQ - Transpose - Reduce sequence.

Ticket: 54249

Code:

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

Validation:

  • Unit tests
  • Framework operation tests
  • Transformation tests
  • Model Optimizer IR Reader check

Documentation:

  • Supported frameworks operations list
  • Guide on how to convert the public model
  • User guide update

@jane-intel jane-intel added bug Something isn't working category: MO Model Optimizer labels May 24, 2021
@jane-intel jane-intel added this to the 2021.4 milestone May 24, 2021
@jane-intel jane-intel requested a review from lazarevevgeny May 24, 2021 09:19
@jane-intel jane-intel requested a review from a team May 24, 2021 09:19
@lazarevevgeny
Copy link
Contributor

@jane-intel , please, add MO code change template and add unit test covering changes.

@lazarevevgeny lazarevevgeny enabled auto-merge (squash) May 25, 2021 08:53
@lazarevevgeny lazarevevgeny merged commit 74293c5 into openvinotoolkit:master May 25, 2021
yekruglov pushed a commit to yekruglov/openvino that referenced this pull request Jun 7, 2021
* Transpose FQ optimization

* Tests added
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
* Transpose FQ optimization

* Tests added
@jane-intel jane-intel deleted the model-optimizer/int/transpose-optimization branch May 26, 2023 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working category: MO Model Optimizer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants