Skip to content

Conversation

@co63oc
Copy link
Contributor

@co63oc co63oc commented Jun 30, 2025

PR Category

Operator Mechanism

PR Types

Bug fixes

Description

PaddleAPITest中是自定义代码
https://github.com/PFCCLab/PaddleAPITest/blob/6576bb1a5e90a1c99322cabcf317411c518eb82e/tester/paddle_to_torch/rules.py#L2528
image

原代码已有修改,只是填充为0,PaddleAPITest测试需要填充为输入的值,所以这里使用tile kernel填充
index 可以结尾维度为0,其他维度为0会有错误,结尾维度为0时,输出shape为 index_shape[:-1] + x_shape,tile中参数对应x_shape部分设置为1
修改tile kernel, tile grad kernel, EigenBroadcastGrad增加支持类型,tile grad 使用 EigenBroadcastGrad

PaddleAPITest 测试通过,错误为numpy error
image
image

@paddle-bot
Copy link

paddle-bot bot commented Jun 30, 2025

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Jun 30, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jun 30, 2025

Codecov Report

Attention: Patch coverage is 40.00000% with 6 lines in your changes missing coverage. Please review.

Please upload report for BASE (develop@a311cbc). Learn more about missing BASE report.

Files with missing lines Patch % Lines
paddle/phi/kernels/cpu/gather_nd_kernel.cc 33.33% 6 Missing ⚠️

❌ Your patch status has failed because the patch coverage (40.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 #73702 +/- ## ========================================== Coverage ? 40.00% ========================================== Files ? 2 Lines ? 10 Branches ? 0 ========================================== Hits ? 4 Misses ? 6 Partials ? 0 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@co63oc co63oc changed the title [0-size Tensor Job2 No.16] Add 0-size Tensor support for paddle.gather_nd [0-size Tensor Job2 No.16、85] Add 0-size Tensor support for paddle.gather_nd Jun 30, 2025
@luotao1 luotao1 added the HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 label Jun 30, 2025
@co63oc
Copy link
Contributor Author

co63oc commented Jul 2, 2025

/re-run all-failed

@DanielSun11
Copy link
Contributor

tile supports [uint8_t int16_t float8_e4m3fn float8_e5m2 int8_t] now, but its grad op kernel not supported.

@co63oc co63oc changed the title [0-size Tensor Job2 No.16、85] Add 0-size Tensor support for paddle.gather_nd [0-size Tensor Job2 No.16、85] Add 0-size Tensor support for paddle.gather_nd [fluid_ops] Jul 2, 2025
@co63oc
Copy link
Contributor Author

co63oc commented Jul 3, 2025

tile supports [uint8_t int16_t float8_e4m3fn float8_e5m2 int8_t] now, but its grad op kernel not supported.

已修改tile grad 和依赖的 EigenBroadcastGrad

@co63oc
Copy link
Contributor Author

co63oc commented Jul 3, 2025

/re-run all-failed

Copy link
Contributor

@DanielSun11 DanielSun11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DanielSun11 DanielSun11 merged commit 4ffa573 into PaddlePaddle:develop Jul 4, 2025
93 of 97 checks passed
@co63oc co63oc deleted the f64 branch July 10, 2025 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务

4 participants