Skip to content

[API compatibility] add paddle.Tensor.ravel #74454

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 5 commits into from
Aug 12, 2025
Merged

[API compatibility] add paddle.Tensor.ravel #74454

merged 5 commits into from
Aug 12, 2025

Conversation

@Starrysea996
Copy link
Contributor

@Starrysea996 Starrysea996 commented Aug 7, 2025 •
edited by zhwesky2010
Loading

PR Category

User Experience

PR Types

New features

Description

add Tensor.ravel api for paddle, similar to torch.Tensor.ravel
@paddle-bot
Copy link

paddle-bot bot commented Aug 7, 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.
zhwesky2010
zhwesky2010 previously approved these changes Aug 7, 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 changed the title [API compatibility] add ravel api [API compatibility] add paddle.Tensor.ravel Aug 7, 2025
@zhwesky2010
Copy link
Contributor

#74439 这个API已经加上了paddle.ravel，这个PR主要就处理 paddle.Tensor.ravel吧，然后单测不用改 都加上去，另外单测要测下paddle.Tensor.ravel
@Starrysea996
Copy link
Contributor Author

单测已经改为paddle.Tensor.ravel
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@489fc16). Learn more about missing BASE report.

⚠️ Current head c8a93b6 differs from pull request most recent head 64cb54c

Please upload reports for the commit 64cb54c to get more accurate results.

Additional details and impacted files 
@@ Coverage Diff @@ ## develop #74454 +/- ## =========================================== Coverage ? 100.00% =========================================== Files ? 1 Lines ? 2 Branches ? 0 =========================================== Hits ? 2 Misses ? 0 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
Copy link
Contributor

@Starrysea996 这个padle.ravel已经合入了，这里新增Tensor.ravel就可以了，不用再增加ravel了
zhwesky2010
zhwesky2010 approved these changes Aug 8, 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 11, 2025 11:26
XiaoguangHu01
XiaoguangHu01 approved these changes Aug 12, 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
jeff41404
jeff41404 approved these changes Aug 12, 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 409515d into PaddlePaddle:develop Aug 12, 2025
91 of 96 checks passed
maxiaolong001 pushed a commit to maxiaolong001/Paddle that referenced this pull request Aug 13, 2025
@Starrysea996 @maxiaolong001
[API compatibility] add paddle.Tensor.ravel (PaddlePaddle#74454)
97725aa 
* add ravel api * use paddle.Tensor.ravel for testcase * replace ravel param x with input * change copyright time * add only Tensor.ravel
This was referenced Aug 25, 2025
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API Compatibility

7 participants

@Starrysea996 @zhwesky2010 @codecov-commenter @jeff41404 @kolinwei @SigureMo @XiaoguangHu01