Skip to content

[Flux] Enabling Flux specific CI #1173

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

[Flux] Enabling Flux specific CI #1173

wants to merge 11 commits into from

Conversation

wwwjn
Copy link
Contributor

@wwwjn wwwjn commented May 7, 2025

Context

  1. Add 2 tests for flux specific changes, and setup CI:
    • Integration tests: Running basic, parallelism, checkpointing and other integration tests.
    • Unite tests: currently only one tests to test flux_dataloader.
  2. Create a small dataset (sample = 10) for offline CI test environment.

Next step:

  • Adding generate image scripts (need basic parallelism), and add a test to generate images.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label May 7, 2025
@wwwjn wwwjn changed the title Flux ci [WIP May 7, 2025
@wwwjn wwwjn changed the title [WIP [WIP][Flux] Enabling Flux specific CI May 7, 2025
@wwwjn wwwjn marked this pull request as draft May 8, 2025 18:26
@wwwjn wwwjn force-pushed the flux-ci branch 2 times, most recently from 0bfc42e to 4671601 Compare May 16, 2025 21:06
@wwwjn wwwjn marked this pull request as ready for review May 16, 2025 21:13
@wwwjn wwwjn requested a review from tianyu-l May 16, 2025 21:13
@wwwjn wwwjn changed the title [WIP][Flux] Enabling Flux specific CI [Flux] Enabling Flux specific CI May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants