-
Notifications
You must be signed in to change notification settings - Fork 5.9k
[Accuracy diff No.168] Fix accuracy (output type) diff for paddle.floor and paddle.ceil API #74598
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
[Accuracy diff No.168] Fix accuracy (output type) diff for paddle.floor and paddle.ceil API #74598
Conversation
|
你的PR提交成功,感谢你对开源项目的贡献! |
|
/re-run all-failed |
1 similar comment
|
/re-run all-failed |
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (83.33%) 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 #74598 +/- ##
==========================================
Coverage ? 83.33%
==========================================
Files ? 3
Lines ? 12
Branches ? 0
==========================================
Hits ? 10
Misses ? 2
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/re-run all-failed |
wanghuancoder
left a comment
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
| Returns: | ||
| Tensor. Output of Floor operator, a Tensor with shape same as input | ||
| (integer types are autocasted into float32). |
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.
记得修改中文文档的这里。
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.
…or and paddle.ceil API (PaddlePaddle#74598) * fix(activation_kernel.cc/cu): fix output type diff for floor/ceil kernel * fix(test_activation_op.py): add unit test * fix(full_kernel.cc/cu): add int8 support for full_like * fix(activation_functor.h): fix floor/ceil functor for int dtype input * fix(test_activation_op.py): add unit test
PR Category
Operator Mechanism
PR Types
Bug fixes
Description
Currently, the
floorandceilkernel exhibits different output types between PyTorch and Paddle. For instance, when the input datatype isint32, PyTorch outputsint32, whereas Paddle outputsfloat32. The same isceil.This is because
floorandceilare conducted type promotion.This PR
pcard-67164