Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[tests] add attention backend tests.#13174

Merged
sayakpaul merged 20 commits into
mainfrom
attn-backend-tests
May 12, 2026
Merged

[tests] add attention backend tests.#13174
sayakpaul merged 20 commits into
mainfrom
attn-backend-tests

Conversation

@sayakpaul

Copy link
Copy Markdown
Member

What does this PR do?

Add attention backend tests.

@sayakpaul sayakpaul requested a review from DN6 February 23, 2026 06:01
@github-actions github-actions Bot added tests size/L PR with diff > 200 LOC labels Apr 17, 2026
Comment thread tests/models/testing_utils/attention.py Outdated
_BF16_REQUIRED_BACKENDS = {
AttentionBackendName._NATIVE_CUDNN,
AttentionBackendName.FLASH_HUB,
AttentionBackendName._FLASH_3_HUB,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Include Sage as well?

Comment thread tests/models/testing_utils/attention.py Outdated
Comment thread tests/models/testing_utils/attention.py Outdated
Comment thread tests/models/testing_utils/attention.py Outdated
Comment thread tests/models/testing_utils/attention.py Outdated
Comment thread tests/models/testing_utils/attention.py Outdated
Comment thread tests/models/testing_utils/attention.py Outdated
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 23, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 23, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 23, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 23, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 23, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 23, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 23, 2026
@sayakpaul

Copy link
Copy Markdown
Member Author

@DN6, all your comments are addressed. LMK if anything is not addressed. I purposefully discarded SAGE to prefer the ones that I feel are more prevalent.

@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 24, 2026
Comment thread tests/models/testing_utils/attention.py
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 24, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 24, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 24, 2026
@github-actions github-actions Bot added size/L PR with diff > 200 LOC and removed size/L PR with diff > 200 LOC labels Apr 25, 2026
@sayakpaul sayakpaul requested a review from DN6 April 25, 2026 00:33
@sayakpaul

Copy link
Copy Markdown
Member Author

@DN6 a gentle ping.

@sayakpaul sayakpaul merged commit cbdedba into main May 12, 2026
15 of 18 checks passed
@sayakpaul sayakpaul deleted the attn-backend-tests branch May 12, 2026 08:04
ADJB1212 pushed a commit to ADJB1212/diffusers that referenced this pull request May 12, 2026
* add attention backend tests.

* remove existing tests/others/test_attention_backends.py file

* modify generate_model_tests.py

* remove native.

* account for _keep_in_fp32_modules

* don't skip when exception is raised.

* use is_kernels_available()

* mark with compile.

* move rtol and atol to methods as defaults.

* Apply suggestions from code review

Co-authored-by: Sayak Paul <[email protected]>

* up

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

Labels

size/L PR with diff > 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants