-
Notifications
You must be signed in to change notification settings - Fork 5.9k
[BUG Fix] Fix cumsum dtype bug #74830
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提交成功,感谢你对开源项目的贡献! |
SigureMo
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.
加一下单测
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #74830 +/- ##
===========================================
Coverage ? 100.00%
===========================================
Files ? 1
Lines ? 6
Branches ? 0
===========================================
Hits ? 6
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
SigureMo
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.
PR Category
Execute Infrastructure
PR Types
Bug fixes
Description
torch 的
cumsum函数,dtype不为 None 时,会指定输出类型#74625 引入后,导致 dtype 非 None 时,只要输入是 int (及其衍生类型),便会自动转化为
int64,显然不合理导致 FastDeploy 运行 ERNIE 4.5 Turbo 系列模型全部挂掉
Paddle/python/paddle/tensor/math.py
Lines 4226 to 4234 in 955e32c
Paddle 测试代码: