refactor autoencoder_kl_cogvideox tests by akshan-main · Pull Request #13840 · huggingface/diffusers · GitHub
Skip to content

refactor autoencoder_kl_cogvideox tests - #13840

Merged
sayakpaul merged 3 commits into
huggingface:mainfrom
akshan-main:tests-refactor-autoencoder-cogvideox
Jun 3, 2026
Merged

sayakpaul merged 3 commits into
huggingface:mainfrom
akshan-main:tests-refactor-autoencoder-cogvideox

Conversation

@akshan-main

Copy link
Copy Markdown
Contributor

What does this PR do?

Part of the ongoing modeling-test migration (following #13369 and #13153). Migrates the AutoencoderKLCogVideoX test suite to the mixin-based structure (Config + ModelTesterMixin / TrainingTesterMixin / MemoryTesterMixin / NewAutoencoderTesterMixin).

Before submitting

Who can review?

@sayakpaul @DN6

@github-actions github-actions Bot added tests size/M PR with diff < 200 LOC labels May 29, 2026

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.

Don't think this attribute is needed.

Comment on lines 84 to 87

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.

I don't think we have to skip this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed both

@akshan-main
akshan-main force-pushed the tests-refactor-autoencoder-cogvideox branch from ff4b0ec to 89da4cc Compare June 2, 2026 02:23
@akshan-main
akshan-main requested a review from DN6 June 2, 2026 08:57
@sayakpaul
sayakpaul merged commit 334ef1a into huggingface:main Jun 3, 2026
13 checks passed
DN6 pushed a commit that referenced this pull request Jul 1, 2026
Co-authored-by: Sayak Paul <spsayakpaul@gmail.com>
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.

4 participants