Skip to content

[CI] fix test_qwen3_5_vl_run_mtp assert msg#1776

Open
nil0x9 wants to merge 1 commit into
InternLM:mainfrom
nil0x9:linty/fix-qwen35-ci-assrt-msg
Open

[CI] fix test_qwen3_5_vl_run_mtp assert msg#1776
nil0x9 wants to merge 1 commit into
InternLM:mainfrom
nil0x9:linty/fix-qwen35-ci-assrt-msg

Conversation

@nil0x9
Copy link
Copy Markdown
Collaborator

@nil0x9 nil0x9 commented May 11, 2026

No description provided.

@nil0x9
Copy link
Copy Markdown
Collaborator Author

nil0x9 commented May 11, 2026

@claude review

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 11, 2026

Claude: LGTM. Trivial one-liner fix — the assertion message was printing the dictionary key (e.g. "text") instead of the expected numeric value (loss_reference[key]). The fix is correct and consistent with how the message should read.

@nil0x9
Copy link
Copy Markdown
Collaborator Author

nil0x9 commented May 12, 2026

@claude rebase and merge it into main

@nil0x9 nil0x9 closed this May 12, 2026
@nil0x9 nil0x9 force-pushed the linty/fix-qwen35-ci-assrt-msg branch from 4e3865e to 287173b Compare May 12, 2026 05:23
@nil0x9 nil0x9 reopened this May 12, 2026
@nil0x9 nil0x9 requested a review from HAOCHENYE May 12, 2026 05:35
@nil0x9 nil0x9 force-pushed the linty/fix-qwen35-ci-assrt-msg branch from e0b6410 to 45d7a4a Compare May 12, 2026 06:59
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.

1 participant