Skip to content

Conversation

@co63oc
Copy link
Contributor

@co63oc co63oc commented Jun 14, 2025

PR Category

Execute Infrastructure

PR Types

Improvements

Description

[0-size Tensor No.6、7、284、285]

paddle.amax
paddle.amin
paddle.Tensor.amax
paddle.Tensor.amin

[0-size Tensor Job2 No.47、49、88、90]
任务二中在同一个单测文件
paddle.max
paddle.min
paddle.Tensor.max
paddle.Tensor.min

修改 paddle/phi/kernels/funcs/reduce_function.h 中检查
修改 amax, amin CPU 反向, max, min 使用的是amax, amin,修改 max,min XPU 反向,以解决CI中报错
infermeta 不用修改

PaddleAPITest 测试
amax
GPU 测试通过
image
CPU 测试通过
image

amin
GPU 测试通过
image
CPU 测试通过
image

max
GPU 测试通过
image
CPU 错误为torch error
image

min
GPU 错误为torch error
image
CPU 错误为numpy error
image

@co63oc co63oc changed the title [0-size Tensor No.6、7、284、285] Add 0-size Tensor support for amin [0-size Tensor No.6、7、284、285] [0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin Jun 14, 2025
@paddle-bot paddle-bot bot added the contributor External developers label Jun 14, 2025
@co63oc
Copy link
Contributor Author

co63oc commented Jun 14, 2025

/re-run all-failed

@codecov-commenter
Copy link

Codecov Report

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

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

Files with missing lines Patch % Lines
paddle/phi/kernels/cpu/reduce_amax_grad_kernel.cc 0.00% 3 Missing ⚠️
paddle/phi/kernels/cpu/reduce_amin_grad_kernel.cc 0.00% 3 Missing ⚠️

❌ 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   #73333   +/-   ##
==========================================
  Coverage           ?    0.00%           
==========================================
  Files              ?        2           
  Lines              ?        6           
  Branches           ?        0           
==========================================
  Hits               ?        0           
  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 No.6、7、284、285] [0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin [0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin Jun 16, 2025
@co63oc co63oc changed the title [0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin [0-size Tensor No.6、7、284、285][0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin Jun 16, 2025
@co63oc co63oc changed the title [0-size Tensor No.6、7、284、285][0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin [0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin Jun 16, 2025
@luotao1 luotao1 added the HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 label Jun 16, 2025
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

@luotao1 luotao1 merged commit b78a40c into PaddlePaddle:develop Jun 17, 2025
80 of 84 checks passed
@co63oc co63oc changed the title [0-size Tensor Job2 No.47、49、88、90]Add 0-size Tensor support for amin [0-size Tensor No.6、7、284、285]Add 0-size Tensor support for amin Jun 17, 2025
@co63oc co63oc deleted the e83 branch June 24, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants