-
Notifications
You must be signed in to change notification settings - Fork 5.7k
[0-size Tensor No.59、62、250] Add 0-size Tensor support for fftshift etc #72809
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 ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #72809 +/- ##
==========================================
Coverage ? 0.00%
==========================================
Files ? 1
Lines ? 3
Branches ? 0
==========================================
Hits ? 0
Misses ? 3
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
已修改fftshift/ifftshift错误,其他算子再提交PR修改,CI已完成 |
@DanielSun11 可以review了 |
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
PR Category
Operator Mechanism
PR Types
Others
Description
59 paddle.fft.fftshift
62 paddle.fft.ifftshift
250 paddle.roll
修改的是 fftshift, ifftshift, fftshift, ifftshift 是cuda错误
fftshift, ifftshift调用的是paddle.roll,所以修改roll kernel
cpu/gpu前向已修改,修改反向和xpu kernel
infermeta不用修改
PaddleAPITest 测试通过


单测参照已有单测编写