Skip to content

[0-size Tensor No.205] Add 0-size Tensor support for pad - #73821

Merged
DanielSun11 merged 6 commits into
PaddlePaddle:developfrom
co63oc:g26
Jul 9, 2025
Merged

DanielSun11 merged 6 commits into
PaddlePaddle:developfrom
co63oc:g26

Conversation

@co63oc

@co63oc co63oc commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

PR Category

Operator Mechanism

PR Types

Bug fixes

Description

[0-size Tensor No.205] Add 0-size Tensor support for pad [fluid_ops]

修改前向和反向,前向填充为pad_value

修改python接口,如果pad为0-size tensor,返回x.clone()

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

@paddle-bot

paddle-bot Bot commented Jul 4, 2025

Copy link
Copy Markdown

你的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 Jul 4, 2025
@codecov-commenter

codecov-commenter commented Jul 5, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@673924e). Learn more about missing BASE report.

Files with missing lines Patch % Lines
paddle/phi/kernels/cpu/pad3d_kernel.cc 25.00% 3 Missing ⚠️
paddle/phi/kernels/cpu/pad3d_grad_kernel.cc 0.00% 1 Missing ⚠️
python/paddle/nn/functional/common.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             develop   #73821   +/-   ##
==========================================
  Coverage           ?   37.50%           
==========================================
  Files              ?        3           
  Lines              ?        8           
  Branches           ?        0           
==========================================
  Hits               ?        3           
  Misses             ?        5           
  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 CI测试不review pad [fluid_ops] [0-size Tensor No.205] Add 0-size Tensor support for pad [fluid_ops] Jul 5, 2025
@co63oc

co63oc commented Jul 6, 2025

Copy link
Copy Markdown
Contributor Author

/re-run all-failed

1 similar comment
@co63oc

co63oc commented Jul 7, 2025

Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@luotao1 luotao1 added the HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 label Jul 7, 2025
@co63oc

co63oc commented Jul 8, 2025

Copy link
Copy Markdown
Contributor Author

/re-run all-failed

DanielSun11
DanielSun11 previously approved these changes Jul 8, 2025

@DanielSun11 DanielSun11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@co63oc

co63oc commented Jul 8, 2025

Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@DanielSun11

Copy link
Copy Markdown
Contributor

image
xpu 有编译错误

@co63oc

co63oc commented Jul 8, 2025

Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@co63oc

co63oc commented Jul 8, 2025

Copy link
Copy Markdown
Contributor Author

image xpu 有编译错误

已修改,现在编译通过,测试是看不到错误信息
image

@DanielSun11

Copy link
Copy Markdown
Contributor

image xpu 有编译错误

已修改,现在编译通过,测试是看不到错误信息 image

单测报错:
image

当无法直接显示日志时可以通过:
image
两种方式获得全部日志

@co63oc

co63oc commented Jul 9, 2025

Copy link
Copy Markdown
Contributor Author

当无法直接显示日志时可以通过: image 两种方式获得全部日志

感谢回复,修改后CI已完成

@DanielSun11 DanielSun11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@co63oc co63oc changed the title [0-size Tensor No.205] Add 0-size Tensor support for pad [fluid_ops] [0-size Tensor No.205] Add 0-size Tensor support for pad Aug 11, 2026
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.

4 participants