Skip to content

Added erase pattern for double redundant transpose ops#992

Open
LeslieXMOS wants to merge 3 commits into
xmos:developfrom
LeslieXMOS:develop
Open

Added erase pattern for double redundant transpose ops#992
LeslieXMOS wants to merge 3 commits into
xmos:developfrom
LeslieXMOS:develop

Conversation

@LeslieXMOS
Copy link
Copy Markdown
Contributor

Added an erase pattern such that redundant transpose ops will be eliminate from the graph

For example
image
The three transpose ops will be eliminated, as they are doing nothing.
This PR will make the final graph like this
image

@henkmuller
Copy link
Copy Markdown
Contributor

That looks great - I am adding Andrew Stanford-Jason for a review

@LeslieXMOS
Copy link
Copy Markdown
Contributor Author

Added three model here to prove this is working and how it will work
model1.tflite.txt
model2.tflite.txt
model3.tflite.txt

  1. model 1 show how bypass work
    image to image

  2. model 2 show how it will only bypass equivant transpose pair
    image to image

  3. model 3 show how good it is
    image to image

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