-
Notifications
You must be signed in to change notification settings - Fork 5.9k
[0-size Tensor Job2 No.38] Add 0-size Tensor support for index_sample #73276
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提交成功,感谢你对开源项目的贡献! |
这个测试case在最新的paddleAPITest accuracy模式下不报[torch error]吗?后续所有的修改都需要进行paddleAPITest accuracy模式的验证。accuracy模式下paddleAPITest会先跑torch,报[paddle error]和[accuracy error]的case是仍然存在问题的,复测不能通过。如果是PaddleAPITest的问题,也请反馈 |
不报[torch error],这是测试时paddleAPITest 在生成随机数据时报的错误,还在跑torch的步骤之前 |
这个问题paddleAPITest已经修复了,请拉取最新代码再跑一遍。现在应该是报numpy error了,请检查是否还有paddle error |
|
DanielSun11
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
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 请 @luotao1 协调云飞Review

PR Category
Execute Infrastructure
PR Types
Improvements
Description
index_sample 修改前向和反向,cpu/gpu/xpu kernel
infermeta 去掉0-size判断,symbolic shape 中没有对应代码
Paddle/paddle/fluid/pir/dialect/operator/interface/infer_symbolic_shape/binary_infer_sym.cc
Line 1141 in d637d5b
PaddleAPITest 测试通过,CPU/GPU错误都为numpy error
