-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[Caffe Frontend] Add support for Permute layer #9157
Conversation
@@ -586,6 +586,27 @@ def test_forward_LRN(): | |||
_test_lrn(data, local_size=3, alpha=2.0, beta=0.5, k=2.0) | |||
|
|||
|
|||
####################################################################### | |||
# Permute | |||
# ----------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alignment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FrozenGene
Sorry, can you specifically point out what's wrong with the code?
There seems to be no problem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 590 and Line 591.
# Permute
# -------
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FrozenGene
Thanks you very much.
I fixed it.
Since #9060 was merged, |
@mshr-h please take a look at the CI problem |
hi @masahi thank you for your reply. |
* Add support for Permute layer * Add test for Permute layer * Fix alignment
* Add support for Permute layer * Add test for Permute layer * Fix alignment
* Add support for Permute layer * Add test for Permute layer * Fix alignment
Added Permute layer support.
Related to #6206, #9060
@fernchen @zhiics @tqchen @FrozenGene
Can you take a look at this PR, please? Thanks!