Skip to content

[0-size Tensor No.90] Add 0-size Tensor support for paddle.index_fill - #73609

Merged
luotao1 merged 1 commit into
PaddlePaddle:developfrom
co63oc:f51
Jun 30, 2025
Merged

luotao1 merged 1 commit into
PaddlePaddle:developfrom
co63oc:f51

Conversation

@co63oc

@co63oc co63oc commented Jun 25, 2025 •

Copy link
Copy Markdown
Contributor

PR Category

Execute Infrastructure

PR Types

Bug fixes

Description

index_fill

测试错误用例index为0-size,修改python中判断index,如果为0-size,inplace返回x,outplace返回x.clone

PaddleAPITest测试通过
image

@luotao1 luotao1 added contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 labels Jun 25, 2025
@codecov-commenter

codecov-commenter commented Jun 28, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@3efb8db). Learn more about missing BASE report.

Files with missing lines Patch % Lines
python/paddle/tensor/manipulation.py 75.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (75.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   #73609   +/-   ##
==========================================
  Coverage           ?   75.00%           
==========================================
  Files              ?        1           
  Lines              ?        4           
  Branches           ?        0           
==========================================
  Hits               ?        3           
  Misses             ?        1           
  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 commented Jun 28, 2025

Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@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

@luotao1

luotao1 commented Jun 30, 2025

Copy link
Copy Markdown
Collaborator

@co63oc 后续该任务的PR模版都改成:

PR Category
Execute Infrastructure

PR Types
Bug fixes

@luotao1
luotao1 merged commit d44be87 into PaddlePaddle:develop Jun 30, 2025
@co63oc

co63oc commented Jun 30, 2025

Copy link
Copy Markdown
Contributor Author

@co63oc 后续该任务的PR模版都改成:

PR Category
Execute Infrastructure

PR Types
Bug fixes

好的

@co63oc
co63oc deleted the f51 branch July 10, 2025 05:41
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