Skip to content

[ROCm][CI] Update GSM8K eval config to use fp8-and-mixed models list (MI355)#37721

Merged
DarkLight1337 merged 1 commit intovllm-project:mainfrom
ROCm:akaratza_fix_lm_eval_mi355_path
Mar 21, 2026
Merged

[ROCm][CI] Update GSM8K eval config to use fp8-and-mixed models list (MI355)#37721
DarkLight1337 merged 1 commit intovllm-project:mainfrom
ROCm:akaratza_fix_lm_eval_mi355_path

Conversation

@AndreasKaratzas
Copy link
Collaborator

Follow-up for:

Updated the GSM8K correctness eval in the AMD CI pipeline to use the models-mi3xx-fp8-and-mixed.txt config list instead of models-mi3xx-fp8.txt. Addresses failure in mi355_2: LM Eval Small Models (B200-MI325)

Motivation: https://buildkite.com/vllm/amd-ci/builds/6721/steps/canvas?sid=019d09d4-713e-4e07-bcb9-9b38689611a0&tab=output

cc @kenroche

@AndreasKaratzas AndreasKaratzas marked this pull request as ready for review March 20, 2026 22:55
@AndreasKaratzas AndreasKaratzas added the ready ONLY add when PR is ready to merge/full CI is needed label Mar 20, 2026
@mergify mergify bot added ci/build rocm Related to AMD ROCm labels Mar 20, 2026
@github-project-automation github-project-automation bot moved this to Todo in AMD Mar 20, 2026
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates a configuration file for the AMD CI pipeline. Specifically, it changes the model list used for the GSM8K correctness evaluation from models-mi3xx-fp8.txt to models-mi3xx-fp8-and-mixed.txt. This change is intended to fix a CI failure. The modification is straightforward and I did not find any issues of high or critical severity.

@AndreasKaratzas
Copy link
Collaborator Author

@DarkLight1337 DarkLight1337 merged commit 02eec7e into vllm-project:main Mar 21, 2026
14 of 15 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in AMD Mar 21, 2026
JartX pushed a commit to JartX/vllm that referenced this pull request Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build ready ONLY add when PR is ready to merge/full CI is needed rocm Related to AMD ROCm

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants