Skip to content

[CI] Bump Transformers version to 5.23.0 - #67502

Merged
vllm-bot merged 9 commits into
vllm-project:mainfrom
hmellor:bump-transformers-5.23
Nov 14, 2026
Merged

vllm-bot merged 9 commits into
vllm-project:mainfrom
hmellor:bump-transformers-5.23

Conversation

@hmellor

@hmellor hmellor commented Nov 13, 2026

Copy link
Copy Markdown
Member

Overview

Bump Transformers to v5.23.0. Follows #64529.

Claims

Validation

The HF PyPI mirror does not have 5.23.0 yet, so the compiled requirements were edited by hand (pin and hashes from pypi.org, committed with SKIP=pip-compile) and this is validated by /ci run nightly. Results to follow.


Pull Request Checklist
  • I used vLLM's /pr-checklist skill. (Mandatory for agents, optional for humans).

  • AI assistance was used during the creation of this PR.

  • Design Fit: Minimizes impact on core components, reuses existing functionality, and justifies added complexity.

  • Testing and Validation: Validates the change and ensures any added tests are meaningful and reliable, with CI coverage or documented CI resource constraints and validation performed outside CI.

  • Code Quality and Style: Keeps code and comments clear and concise, and updates relevant documentation and examples.

  • Pull Request Contents: Includes a brief summary and relevant links, supports claims with evidence, explains root causes and implementation trade-offs, and follows the contributing guide.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
@hmellor

hmellor commented Nov 13, 2026

Copy link
Copy Markdown
Member Author

/ci run nightly

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #102497 for commit 934010da13c9.

@mergify mergify Bot added ci/build cpu Related to CPU backends labels Nov 13, 2026
@hmellor
hmellor marked this pull request as ready for review November 13, 2026 15:11
@hmellor
hmellor requested a review from a team November 13, 2026 15:11

@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.

hmellor and others added 5 commits November 13, 2026 18:26
Transformers v5.23 validates that placeholder counts match the passed
media. Qwen3-VL, Gemma 4 and GLM-4.1V process videos separately, then
called the HF processor with text still containing video placeholders.
Rebuild that text without videos, and drop the code that expanded video
tokens in the text or `input_ids`, since vLLM discards that output.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Without it, audio extracted from videos is counted as standalone audio
and fails the placeholder count validation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Transformers v5.23 validates flat kwargs against the processor's kwargs
`TypedDict`, so remote code that omits `total=False` (e.g. MiDashengLM)
makes every kwarg required. Mark these as optional when the remote code
is imported, and warn so the remote code can be fixed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
The remote `BailingMoeV3VL` processor's `__call__` is incompatible with
the kwargs returned by `_merge_kwargs` in Transformers v5.23. Call its
image processor directly, as is already done for videos.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Transformers v5.23 dropped `truncation: False` from the Qwen3-ASR audio
defaults, so `WhisperFeatureExtractor` checkpoints truncate audio longer
than 30 s in HF but not in vLLM.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
@hmellor

hmellor commented Nov 13, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ The previous CI build is still running: https://buildkite.com/vllm/ci/builds/102497

@mergify mergify Bot added multi-modality Related to multi-modality (#4194) qwen Related to Qwen models glm labels Nov 13, 2026
@AndreasKaratzas

AndreasKaratzas commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Could you run the full CI (/ci run all or a nightly) on the latest head, or manually unblock the remaining jobs, so the multimodal groups validate the Transformers 5.23 fixes before merge?

@hmellor

hmellor commented Nov 13, 2026

Copy link
Copy Markdown
Member Author

The initial run was nightly. I'm using retry to avoid being wasteful while burning down the errors. Then the last run when I think it'll all pass will be nightly too

@hmellor

hmellor commented Nov 13, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #102660 for commit 396fb68e5e6f, running 58 failed step(s) from Buildkite CI #102497.

@hmellor

hmellor commented Nov 14, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ Queued 13 failed job(s) for retry in Buildkite CI #102660.

@khluu khluu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ci stamp

@hmellor hmellor mentioned this pull request Nov 14, 2026
6 tasks done
hmellor and others added 2 commits November 14, 2026 11:46
The remote code also imports BASE_VIDEO_PROCESSOR_DOCSTRING, which
Transformers v5.15 removed, so the model cannot load on the vLLM side
either.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
@hmellor

hmellor commented Nov 14, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #102766 for commit 41a62c95f335, running 12 failed step(s) from Buildkite CI #102660.

@hmellor

hmellor commented Nov 14, 2026

Copy link
Copy Markdown
Member Author

/ci run nightly

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #102797 for commit 21193f8cb32b.

@hmellor

hmellor commented Nov 14, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ Queued 2 failed job(s) for retry in Buildkite CI #102797.

@vllm-bot
vllm-bot merged commit 0edc2ea into vllm-project:main Nov 14, 2026
715 of 718 checks passed
@hmellor
hmellor deleted the bump-transformers-5.23 branch November 14, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build cpu Related to CPU backends glm multi-modality Related to multi-modality (#4194) qwen Related to Qwen models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants