Skip to content

[CI] Refactor Skyreels, Lumina, Ominigen, Mochi transformer tests#13348

Merged
DN6 merged 6 commits into
mainfrom
skyreels-mochi-lumina-omnigen
Jun 9, 2026
Merged

[CI] Refactor Skyreels, Lumina, Ominigen, Mochi transformer tests#13348
DN6 merged 6 commits into
mainfrom
skyreels-mochi-lumina-omnigen

Conversation

@DN6

@DN6 DN6 commented Mar 26, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

Fixes # (issue)

Before submitting

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@DN6 DN6 requested a review from sayakpaul March 27, 2026 07:50
from ..testing_utils import (
BaseModelTesterConfig,
ModelTesterMixin,
TorchCompileTesterMixin,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe compilation tests are not needed?

from ..testing_utils import (
BaseModelTesterConfig,
ModelTesterMixin,
TorchCompileTesterMixin,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same as above.

from ..testing_utils import (
BaseModelTesterConfig,
ModelTesterMixin,
TorchCompileTesterMixin,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same as above.

from ..testing_utils import (
BaseModelTesterConfig,
ModelTesterMixin,
TorchCompileTesterMixin,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same as above.

from ..testing_utils import (
BaseModelTesterConfig,
ModelTesterMixin,
TorchCompileTesterMixin,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same as above.

@github-actions github-actions Bot added tests size/L PR with diff > 200 LOC labels Apr 14, 2026
@DN6 DN6 requested a review from sayakpaul June 4, 2026 09:14
@sayakpaul

Copy link
Copy Markdown
Member

Should we keep the LoRA tests where relevant? I feel like that should be ensured?

@DN6

DN6 commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator Author

Should we keep the LoRA tests where relevant? I feel like that should be ensured?

@sayakpaul
For which model here? I think usage across the board is fairly low.

@sayakpaul

Copy link
Copy Markdown
Member

If usage is considered, then okay makes sense to not do LoRA.

@DN6 DN6 merged commit c07f09c into main Jun 9, 2026
13 of 14 checks passed
DN6 added a commit that referenced this pull request Jul 1, 2026
…3348)

* update

* update

---------

Co-authored-by: Sayak Paul <spsayakpaul@gmail.com>
@sayakpaul sayakpaul deleted the skyreels-mochi-lumina-omnigen branch July 3, 2026 03:49
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