Skip to content

Conversation

@sayakpaul
Copy link
Member

What does this PR do?

As the title mentions. Some changes:

  • Removes push_tests_fasts.yml
  • Remove the compiler tester mixin from SkyReelsV2Transformer3DTests as that's not how it's supposed to be tested.
  • Marks the compilation tests (including hotswapping tests) with is_torch_compile.
  • Removes run_torch_compile_tests from the push_tests and ensures that the compilation-related tests only run nightly.

pytest -m "is_torch_compile" tests/ --collect-only -q > collected_tests.txt gives me exactly the kinds of tests we wanna run for compilation. I have verified it.

@sayakpaul sayakpaul requested review from DN6 and dg845 October 21, 2025 05:55
@sayakpaul
Copy link
Member Author

@stevhliu, the doc-related error on the CI seems familiar to you?

@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

@github-actions github-actions bot added the stale Issues that haven't received updates label Jan 9, 2026
@stevhliu
Copy link
Member

stevhliu commented Jan 9, 2026

hi, the doc error should be fixed in #12522

@github-actions github-actions bot removed the stale Issues that haven't received updates label Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants