Skip to content

[CI] Add gemma-4-E4B-it-assistant to CI gsm8k for GemmaMTP - #49243

Merged
tlrmchlsmth merged 2 commits into
vllm-project:mainfrom
neuralmagic:add-gemma-4-E4B-it-assistant-gsm8k
Jul 21, 2026
Merged

tlrmchlsmth merged 2 commits into
vllm-project:mainfrom
neuralmagic:add-gemma-4-E4B-it-assistant-gsm8k

Conversation

@mgoin

@mgoin mgoin commented Jul 20, 2026

Copy link
Copy Markdown
Member

Purpose

To better cover against regression like #47953

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.

Signed-off-by: mgoin <mgoin64@gmail.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@mgoin mgoin added ready ONLY add when PR is ready to merge/full CI is needed ci/build labels Jul 21, 2026
@tlrmchlsmth
tlrmchlsmth merged commit b8fb56d into vllm-project:main Jul 21, 2026
19 of 21 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants