Skip to content

[serge] Fix 2 integration tests for model qwen2_5_omni failing with output_mismatch (list output differs (2))#47541

Open
sergereview[bot] wants to merge 1 commit into
mainfrom
serge/fix/itf-f8da0434e923-26c00b1f
Open

[serge] Fix 2 integration tests for model qwen2_5_omni failing with output_mismatch (list output differs (2))#47541
sergereview[bot] wants to merge 1 commit into
mainfrom
serge/fix/itf-f8da0434e923-26c00b1f

Conversation

@sergereview

@sergereview sergereview Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

CI

Original CI failure

  • Failure group: 1/1: 2 integration tests for model qwen2_5_omnifailing withoutput_mismatch (list output differs (2))
  • tests/models/qwen2_5_omni/test_modeling_qwen2_5_omni.py::Qwen2_5OmniModelIntegrationTest::test_small_model_integration_test_batch [multi-gpu] (output_mismatch, seen 7/7)

Relates to #47538

The test_small_model_integration_test_batch integration test for qwen2_5_omni was failing on GPU because the hard-coded CUDA 8 expected decoded texts no longer match the model's current greedy generation output.

Root cause: the Expectations entry for ("cuda", 8) expected "The sound is a glass shattering. The dog in the picture is a Labrador Retriever.", but the model now generates "The sound is glass shattering, and the dog is a Labrador Retriever." for the exercised batch configuration.

This patch updates the ("cuda", 8) expected strings in the batch test to the newly observed output, keeping the test's existing Expectations style and leaving unrelated model code untouched.


This change was produced automatically by serge from a CI failure report. The patch was generated by an LLM and applied by serge; review before merging.

serge v0.1.0 · model: moonshotai/Kimi-K2.7-Code · 64 LLM turns · 63 tool calls · 276.7s · 2095331 in / 12020 out tokens

… `output_mismatch` (list output differs (2))
@sergereview
sergereview Bot marked this pull request as ready for review July 25, 2026 08:35
@github-actions
github-actions Bot requested a review from ydshieh July 25, 2026 08:35
@github-actions

Copy link
Copy Markdown
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: qwen2_5_omni

@tarekziade

Copy link
Copy Markdown
Collaborator

run-slow: qwen2_5_omni

@github-actions

Copy link
Copy Markdown
Contributor

Workflow Run ⚙️

This comment contains run-slow, running the specified jobs:

models: ["models/qwen2_5_omni"]
quantizations: []

@github-actions

Copy link
Copy Markdown
Contributor

CI recap

Dashboard: View test results in Grafana
Latest run: 30151379409
Result: success | Grafana metrics are not available yet.

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@github-actions

Copy link
Copy Markdown
Contributor

CI Results

Workflow Run ⚙️

Commit Info

Context Commit Description
RUN d251c2f6 workflow commit (merge commit)
PR 76685833 branch commit (from PR)
main b6d5084f base commit (on main)

✅ No failing test specific to this PR 🎉 👏 !

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants