Skip to content

supports fa3_varlen api #72805

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 10 commits into from
May 30, 2025
Merged

supports fa3_varlen api #72805

merged 10 commits into from
May 30, 2025

Conversation

@carryyu
Copy link
Contributor

@carryyu carryyu commented May 20, 2025 •
edited
Loading

PR Category

Inference

PR Types

Others

Description

card-71500
supports fa3_varlen api

@paddle-bot
Copy link

paddle-bot bot commented May 20, 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.

@CLAassistant
Copy link

CLAassistant commented May 20, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLAassistant commented May 20, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

zyfncg
zyfncg reviewed May 21, 2025
View reviewed changes
zyfncg
zyfncg previously approved these changes May 22, 2025
View reviewed changes
SigureMo
SigureMo reviewed May 22, 2025
View reviewed changes
SigureMo
SigureMo reviewed May 22, 2025
View reviewed changes
qingqing01
qingqing01 reviewed May 23, 2025
View reviewed changes
Copy link
Contributor

@qingqing01 qingqing01 left a comment

Choose a reason for hiding this comment

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

need to add UT.

@carryyu
Copy link
Contributor Author

carryyu commented May 29, 2025

/re-run api-benchmark

@carryyu
Copy link
Contributor Author

carryyu commented May 29, 2025

need to add UT.

已添加

@carryyu
Copy link
Contributor Author

carryyu commented May 30, 2025

/re-run xpu test

@carryyu
Copy link
Contributor Author

carryyu commented May 30, 2025

/re-run xpu

@carryyu carryyu merged commit c0032d7 into PaddlePaddle:develop May 30, 2025
161 of 169 checks passed
umiswing added a commit to umiswing/Paddle that referenced this pull request Jul 4, 2025
@umiswing
Revert "supports fa3_varlen api (PaddlePaddle#72805)"
b81a1d2 
This reverts commit c0032d7.
umiswing added a commit to umiswing/Paddle that referenced this pull request Jul 7, 2025
@umiswing
Add fa3 varlen
f4acfb1 
Revert "supports fa3_varlen api (PaddlePaddle#72805)"

This reverts commit c0032d7.

support fa3 varlen

rename ctx to dev_ctx

rename

fix comment, add doc

rename infermeta

rename

add flash_attn_varlen_func
GuoxiaWang pushed a commit that referenced this pull request Jul 30, 2025
@umiswing
support fa3 varlen bwd and modify fa3 varlen fwd api (#73831)
35cae23 
* Revert "supports fa3_varlen api (#72805)"

This reverts commit c0032d7.

* support fa3 varlen

* rename ctx to dev_ctx

* rename

* fix comment, add doc

* rename infermeta

* rename

* add flash_attn_varlen_func

* fix check

* add init
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sunzhongkai588 sunzhongkai588 sunzhongkai588 left review comments

@qingqing01 qingqing01 qingqing01 approved these changes

@zyfncg zyfncg zyfncg approved these changes

@SigureMo SigureMo SigureMo approved these changes

Assignees

No one assigned

Labels

skip-ci: api-benchmark skip-ci: approval

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@carryyu @CLAassistant @qingqing01 @zyfncg @SigureMo @sunzhongkai588 @swgu98