Skip to content

refactor sana transformer tests#13826

Merged
sayakpaul merged 3 commits into
huggingface:mainfrom
akshan-main:tests-refactor-sana
May 29, 2026
Merged

refactor sana transformer tests#13826
sayakpaul merged 3 commits into
huggingface:mainfrom
akshan-main:tests-refactor-sana

Conversation

@akshan-main
Copy link
Copy Markdown
Contributor

@akshan-main akshan-main commented May 28, 2026

What does this PR do?

Part of the ongoing modeling-test migration (following #13369 and #13153). Refactors the Sana transformer tests into a SanaTransformerTesterConfig plus per-component test classes: ModelTesterMixin, MemoryTesterMixin, TrainingTesterMixin, and AttentionTesterMixin.

SanaTransformer2DModel doesn't set _repeated_blocks, so the compile mixin is not included.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline?
  • Did you read our philosophy doc (important for complex PRs)?
  • Was this discussed/approved via a GitHub issue or the forum? Please add a link to it if that's the case. (Also discussed on slack)
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

@sayakpaul

@github-actions github-actions Bot added size/M PR with diff < 200 LOC tests and removed size/M PR with diff < 200 LOC labels May 28, 2026
@github-actions github-actions Bot added the size/M PR with diff < 200 LOC label May 29, 2026
@sayakpaul sayakpaul merged commit e3fc602 into huggingface:main May 29, 2026
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M PR with diff < 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants