Skip to content

[ci] feat: implement a bot to suggest running tests before merging#14099

Open
sayakpaul wants to merge 2 commits into
mainfrom
suggest-tests
Open

[ci] feat: implement a bot to suggest running tests before merging#14099
sayakpaul wants to merge 2 commits into
mainfrom
suggest-tests

Conversation

@sayakpaul

Copy link
Copy Markdown
Member

Example comment:

Suggested slow tests to run before merge:

- tests/lora/
- tests/models/transformers/
- tests/pipelines/flux/

Where the diff in a PR maps cleanly to core modules and their corresponding tests (peft.py -> LoRA-related tests), we use a deterministic system for the suggestion. However, when that gets convoluted, we also use Qwen3.5-35B-A3 model through HF Inference Provider to rank and polish the suggestion.

@sayakpaul sayakpaul requested a review from yiyixuxu July 1, 2026 11:56
@github-actions github-actions Bot added CI size/L PR with diff > 200 LOC labels Jul 1, 2026
@yiyixuxu yiyixuxu requested a review from DN6 July 6, 2026 22:56
@yiyixuxu

yiyixuxu commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

@DN6 can you take a look at this PR?

down the road, we can streamline the workflow so final review will recommend a couple of tests in the serge UI, and the maintainer can just check a checkbox to run the slow test etc (not in the scope of this PR)

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

Labels

CI size/L PR with diff > 200 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants