Skip to content

Conversation

@LittleHeroZZZX
Copy link
Contributor

@LittleHeroZZZX LittleHeroZZZX commented Aug 12, 2025

PR Category

User Experience

PR Types

New features

Description

添加如下别名并支持 out 参数。
paddle.cat
paddle.clamp
paddle.ger
paddle.take_along_dim
paddle.linalg.matmul
paddle.special.logsumexp
paddle.concatenate

其中,logsumexp 和 matmul 没有兼容 torch,需要做参数别名映射,C++ 下沉逻辑复杂,由 @DanielSun11 负责。

@paddle-bot
Copy link

paddle-bot bot commented Aug 12, 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 changed the title [API Compatibility] Add 7 alias [API Compatibility] Add 7 API alias Aug 12, 2025
@LittleHeroZZZX
Copy link
Contributor Author

/re-run all-failed

zhwesky2010
zhwesky2010 previously approved these changes Aug 14, 2025
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 14, 2025 09:36
@LittleHeroZZZX
Copy link
Contributor Author

/re-run all-failed

SigureMo
SigureMo previously approved these changes Aug 14, 2025
@zhwesky2010 zhwesky2010 self-requested a review August 14, 2025 12:05
@LittleHeroZZZX LittleHeroZZZX dismissed stale reviews from SigureMo and zhwesky2010 via 76433aa August 15, 2025 07:47
SigureMo
SigureMo previously approved these changes Aug 15, 2025
@codecov-commenter
Copy link

codecov-commenter commented Aug 15, 2025

Codecov Report

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

Additional details and impacted files
@@             Coverage Diff             @@
##             develop    #74569   +/-   ##
===========================================
  Coverage           ?   100.00%           
===========================================
  Files              ?         5           
  Lines              ?        14           
  Branches           ?         0           
===========================================
  Hits               ?        14           
  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.

@LittleHeroZZZX LittleHeroZZZX marked this pull request as draft August 18, 2025 02:05
@LittleHeroZZZX LittleHeroZZZX marked this pull request as ready for review August 20, 2025 08:41
zhwesky2010
zhwesky2010 previously approved these changes Aug 20, 2025
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
Copy link
Contributor

CI有些失败的,还需要看看

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

zhwesky2010 commented Aug 20, 2025

@LittleHeroZZZX API-Benchamark问题,看看是不是要换个装饰器,这个性能不高

@LittleHeroZZZX
Copy link
Contributor Author

LittleHeroZZZX commented Aug 20, 2025

@LittleHeroZZZX API-Benchamark问题,看看是不是要换个装饰器,这个性能不高

我只在 clip 上加了装饰器,挂了的这几个 API 我都没动,应该是之前已经合入的 PR 导致的吧?多 rerun 几次估计能过。
另外,API-Benchamark 的基准会每日更新嘛?不更新的话已经合入的几个 API 应该会随机挂?

@LittleHeroZZZX
Copy link
Contributor Author

/re-run all-failed

2 similar comments
@LittleHeroZZZX
Copy link
Contributor Author

/re-run all-failed

@LittleHeroZZZX
Copy link
Contributor Author

/re-run all-failed

SigureMo
SigureMo previously approved these changes Aug 21, 2025
XiaoguangHu01
XiaoguangHu01 previously approved these changes Aug 21, 2025
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

@LittleHeroZZZX
Copy link
Contributor Author

/re-run all-failed

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

@LittleHeroZZZX
Copy link
Contributor Author

/re-run all-failed

@xiaoguoguo626807 xiaoguoguo626807 merged commit bcda69d into PaddlePaddle:develop Aug 25, 2025
91 of 94 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants