Skip to content

Fix incorrect working UnpackPackReverseInputChannels for centernet#9201

Merged
rkazants merged 16 commits intoopenvinotoolkit:masterfrom
evolosen:mo/bug_in_unpack_pack_reverse_input_channels
Dec 27, 2021
Merged

Fix incorrect working UnpackPackReverseInputChannels for centernet#9201
rkazants merged 16 commits intoopenvinotoolkit:masterfrom
evolosen:mo/bug_in_unpack_pack_reverse_input_channels

Conversation

@evolosen
Copy link
Copy Markdown
Contributor

@evolosen evolosen commented Dec 14, 2021

Root cause analysis: Incorrect order in UnpackPackReverseInputChannels as result incorrect inference

Solution: change order

Ticket: CVS-73731

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

@evolosen evolosen requested a review from a team December 14, 2021 08:32
@evolosen evolosen requested review from a team, pavel-esir, vgavrilo and yekruglov and removed request for a team December 22, 2021 10:36
Copy link
Copy Markdown
Collaborator

@rkazants rkazants left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please implement transformation test. The other looks good

@evolosen evolosen requested a review from rkazants December 24, 2021 07:10
@rkazants rkazants merged commit fa1b59b into openvinotoolkit:master Dec 27, 2021
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.

4 participants