Skip to content

[API Compatibility] add output_size parameter to paddle.repeat_interleave and paddle.Tensor.repeat_interleave #74631

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 19 commits into from
Aug 29, 2025
Merged

[API Compatibility] add output_size parameter to paddle.repeat_interleave and paddle.Tensor.repeat_interleave #74631

merged 19 commits into from
Aug 29, 2025

Conversation

aquagull
Copy link
Contributor

@aquagull aquagull commented Aug 15, 2025 •
edited by zhwesky2010
Loading

PR Category

User Experience

PR Types

New features

Description

pcard-71500

给repeat_interleave增加output_size参数，以及增加input->x、dims->axis的参数映射。

image

@paddle-bot Paddle-bot
Copy link

paddle-bot bot commented Aug 15, 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.

@aquagull
Copy link
Contributor Author

/re-run all-failed

zhwesky2010
zhwesky2010 reviewed Aug 18, 2025
View reviewed changes
@codecov-commenter
Copy link

codecov-commenter commented Aug 18, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 62.16216% with 14 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@8b5c009). Learn more about missing BASE report.

Files with missing lines Patch % Lines
.../interface/infer_symbolic_shape/unary_infer_sym.cc 0.00% 6 Missing ⚠️
...interface/infer_symbolic_shape/binary_infer_sym.cc 0.00% 5 Missing ⚠️
python/paddle/tensor/manipulation.py 0.00% 2 Missing ⚠️
paddle/phi/infermeta/binary.cc 66.66% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (62.16%) 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   #74631   +/-   ##
==========================================
  Coverage           ?   62.16%           
==========================================
  Files              ?        7           
  Lines              ?       37           
  Branches           ?        0           
==========================================
  Hits               ?       23           
  Misses             ?       14           
  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.

zhwesky2010
zhwesky2010 reviewed Aug 19, 2025
View reviewed changes
Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

给ops.yaml加参数时，还需要改一下这个文件，做一下patch

@aquagull
Copy link
Contributor Author

/re-run all-failed

zhwesky2010
zhwesky2010 previously approved these changes Aug 20, 2025
View reviewed changes
Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhwesky2010 zhwesky2010 requested a review from SigureMo August 20, 2025 10:25
SigureMo
SigureMo previously approved these changes Aug 20, 2025
View reviewed changes
XiaoguangHu01
XiaoguangHu01 previously approved these changes Aug 20, 2025
View reviewed changes
Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhwesky2010
Copy link
Contributor

@aquagull CI还有其他问题吗

@aquagull aquagull dismissed stale reviews from XiaoguangHu01, SigureMo, and zhwesky2010 via 1502eef August 20, 2025 14:53
@aquagull
Copy link
Contributor Author

@aquagull CI还有其他问题吗

好像是的，coverage挂了，但不太像这个pr的问题，我拉下dev重新跑

@zhwesky2010
Copy link
Contributor

@aquagull CI尽快通过吧，这个PR时间比较长了

@aquagull
Copy link
Contributor Author

/re-run distribute-stable

@aquagull
Copy link
Contributor Author

/re-run all-failed

1 similar comment
@aquagull
Copy link
Contributor Author

/re-run all-failed

zhwesky2010
zhwesky2010 previously approved these changes Aug 27, 2025
View reviewed changes
SigureMo
SigureMo previously approved these changes Aug 27, 2025
View reviewed changes
zyfncg
zyfncg previously approved these changes Aug 27, 2025
View reviewed changes
@aquagull aquagull dismissed stale reviews from zyfncg, SigureMo, and zhwesky2010 via 0034e9c August 28, 2025 07:00
@aquagull
Copy link
Contributor Author

/re-run all-failed

@zhwesky2010 zhwesky2010 requested a review from qingqing01 August 29, 2025 06:50
@zhwesky2010 zhwesky2010 requested a review from jeff41404 August 29, 2025 08:08
jeff41404
jeff41404 approved these changes Aug 29, 2025
View reviewed changes
Copy link
Contributor

@jeff41404 jeff41404 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhwesky2010 zhwesky2010 merged commit c29e57e into PaddlePaddle:develop Aug 29, 2025
165 of 176 checks passed
@aquagull aquagull mentioned this pull request Aug 29, 2025
Merged
@aquagull aquagull deleted the repeat_interleave branch September 12, 2025 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeff41404 jeff41404 jeff41404 approved these changes

@heavengate heavengate heavengate approved these changes

@zyfncg zyfncg zyfncg approved these changes

@yuanlehome yuanlehome yuanlehome approved these changes

@changeyoung98 changeyoung98 changeyoung98 approved these changes

@SigureMo SigureMo SigureMo approved these changes

@zhwesky2010 zhwesky2010 zhwesky2010 approved these changes

@XiaoguangHu01 XiaoguangHu01 Awaiting requested review from XiaoguangHu01

@qingqing01 qingqing01 Awaiting requested review from qingqing01

Assignees
No one assigned
Labels
API Compatibility
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@aquagull @codecov-commenter @zhwesky2010 @jeff41404 @heavengate @zyfncg @yuanlehome @changeyoung98 @SigureMo @XiaoguangHu01