Skip to content

[diffusion] CI: send diffusion nightly-test outputs to slack for correctness monitoring#13784

Closed
yhyang201 wants to merge 11 commits intosgl-project:mainfrom
yhyang201:slack-bot
Closed

[diffusion] CI: send diffusion nightly-test outputs to slack for correctness monitoring#13784
yhyang201 wants to merge 11 commits intosgl-project:mainfrom
yhyang201:slack-bot

Conversation

@yhyang201
Copy link
Collaborator

@yhyang201 yhyang201 commented Nov 23, 2025

Motivation

This PR adds the ability to push diffusion-related performance test outputs to Slack for easier monitoring of generation correctness.
It introduces a new upload_file_to_slack utility and integrates it into the video, image, and image-edit generation flows. For each test case, the generated outputs (and the original image when applicable) are saved temporarily, uploaded to a designated Slack channel along with case ID, model, and prompt metadata, and then cleaned up. This enables clear visualization of test outputs directly in Slack, improving oversight and troubleshooting of generation quality.

Modifications

Accuracy Tests

Benchmarking and Profiling

Checklist

@github-actions github-actions bot added the diffusion SGLang Diffusion label Nov 23, 2025
@yhyang201
Copy link
Collaborator Author

/tag-and-rerun-ci

@mickqian mickqian changed the title diffsuion: send diffusion test outputs to slack for correctness monitoring [diffsuion] CI: send diffusion nightly-test outputs to slack for correctness monitoring Nov 24, 2025
@mickqian mickqian changed the title [diffsuion] CI: send diffusion nightly-test outputs to slack for correctness monitoring [diffusion] CI: send diffusion nightly-test outputs to slack for correctness monitoring Nov 24, 2025
@mickqian mickqian closed this Nov 24, 2025
@yhyang201
Copy link
Collaborator Author

Move to #13833

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

diffusion SGLang Diffusion run-ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants