Skip to content

Conversation

@co63oc
Copy link
Contributor

@co63oc co63oc commented Jun 16, 2025

PR Category

Execute Infrastructure

PR Types

Improvements

Description

[0-size Tensor Job2 No.105] Add 0-size Tensor support for roi_align

输出的shape为 [box[0], input[1], pooled_height, pooled_width],pooled_height, pooled_width 是都要大于0,所以判断input和box的numel

PaddleAPITest 测试,
使用--accuracy 选项,是torch error异常退出
image

使用--paddle_only 选项测试通过,错误为numpy error
image

@co63oc co63oc changed the title CI测试不review roi_align [0-size Tensor Job2 No.105] Add 0-size Tensor support for roi_align Jun 16, 2025
@luotao1 luotao1 added contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 labels Jun 16, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jun 16, 2025

Codecov Report

❌ Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@1f0c5cf). Learn more about missing BASE report.

Files with missing lines Patch % Lines
paddle/phi/kernels/cpu/roi_align_grad_kernel.cc 0.00% 4 Missing ⚠️
paddle/phi/kernels/cpu/roi_align_kernel.cc 33.33% 2 Missing ⚠️

❌ Your patch status has failed because the patch coverage (14.28%) 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 #73361 +/- ## ========================================== Coverage ? 14.28% ========================================== Files ? 2 Lines ? 7 Branches ? 0 ========================================== Hits ? 1 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.
@paddle-bot
Copy link

paddle-bot bot commented Jun 16, 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.

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 cb63efd into PaddlePaddle:develop Jun 17, 2025
50 of 52 checks passed
@co63oc co63oc deleted the f10 branch June 30, 2025 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

5 participants