Skip to content

Fixed L2NormFusing transformation#4526

Merged
lazarevevgeny merged 3 commits intoopenvinotoolkit:masterfrom
lazarevevgeny:l2normalize_pattern_fix
Mar 1, 2021
Merged

Fixed L2NormFusing transformation#4526
lazarevevgeny merged 3 commits intoopenvinotoolkit:masterfrom
lazarevevgeny:l2normalize_pattern_fix

Conversation

@lazarevevgeny
Copy link
Copy Markdown
Contributor

Description: Fixed L2NormToNorm transformation. Updated condition when the L2NormToNorm should be applied. It should work when normalization is applied over channels dimension only or over "C + spatial" dimensions. Solution verified with the TF layer tests (see draft MR 692).

Ticket: 49602

Code:

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

Validation:

  • Unit tests
  • Framework operation tests: draft MR 692
  • Transformation tests
  • Model Optimizer IR Reader check

Documentation:

  • Supported frameworks operations list: N/A no new op enabled
  • Guide on how to convert the public model: N/A no new model enabled
  • User guide update: N/A no user visible changes

@lazarevevgeny lazarevevgeny requested review from a team, mvafin, rkazants and sadolini and removed request for a team February 27, 2021 11:29
@openvino-pushbot openvino-pushbot added the category: MO Model Optimizer label Feb 27, 2021
@lazarevevgeny lazarevevgeny requested a review from mvafin March 1, 2021 07:33
@lazarevevgeny lazarevevgeny changed the title Fixed L2NormFusing transformastion Fixed L2NormFusing transformation Mar 1, 2021
@lazarevevgeny lazarevevgeny merged commit e065cf5 into openvinotoolkit:master Mar 1, 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.

6 participants