-
Notifications
You must be signed in to change notification settings - Fork 5.8k
[API Compatibility] Add paddle.permute、paddle.Tensor.permute #74525
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
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #74525 +/- ##
==========================================
Coverage ? 92.85%
==========================================
Files ? 2
Lines ? 14
Branches ? 0
==========================================
Hits ? 13
Misses ? 1
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
python/paddle/tensor/linalg.py
Outdated
) -> Tensor: ... | ||
|
||
|
||
@overload |
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.
可变参数的用法参考 SizeArgsDecorator 实现吧,API签名还是正常写 |
) -> tuple[tuple[Any, ...], dict[str, Any]]: | ||
if len(args) >= 2 and isinstance(args[1], int): | ||
kwargs[self.var] = list(args[1:]) | ||
args = args[0] |
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.
对于paddle.permute(x, 1, 2, 0)
是处理成 args=1, kwargs={'dims', [2, 0]} 吗,感觉不太对,应该是整个 args都被处理成list吧,list(args)
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.
/re-run all-failed |
1 similar comment
/re-run all-failed |
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.
LGTM
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.
LGTM
53fee70
/re-run all-failed |
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.
LGTM
…addle#74525) * add permute api * fix * code-style * fix * update * update * update * code style * fix doc * fix --------- Co-authored-by: aquagull <[email protected]>
PR Category
User Experience
PR Types
Others
Description
paddle.permute
paddle.Tensor.permute
pcard-71500