Skip to content

[MM][CG][BugFix] Fix Ernie-4.5-VL encoder CG postprocess for multi-path outputs - #51461

Merged
khluu merged 1 commit into
vllm-project:mainfrom
qyYue1389:ernie45vl-postprocess-multipath-fix
Aug 11, 2026
Merged

[MM][CG][BugFix] Fix Ernie-4.5-VL encoder CG postprocess for multi-path outputs#51461
khluu merged 1 commit into
vllm-project:mainfrom
qyYue1389:ernie45vl-postprocess-multipath-fix

Conversation

@qyYue1389

@qyYue1389 qyYue1389 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Purpose

Fix-forward for the crash that prompted #51263 (revert of #45254), instead of reverting.

SupportsEncoderCudaGraph.postprocess_encoder_output now receives outputs: dict[str, torch.Tensor] keyed by encoder path (multi-path graph support), but Ernie4_5_VLMoeForConditionalGeneration's override still treated
the first arg as a single tensor:
File "vllm/model_executor/models/ernie45_vl.py", line 1734, in postprocess_encoder_output grid_thw = batch_mm_kwargs["image_grid_thw"].to(output.device) AttributeError: 'dict' object has no attribute 'device'

#45254 and the multi-path framework change merged around the same time, so each was green independently but the combination broke encoder CUDA-graph replay for Ernie-4.5-VL on main.

Test

pytest tests/models/multimodal/generation/test_vit_cudagraph.py::test_vit_cudagraph_image[ernie45_vl]
— the same case reported in #51263 (build 82629).

Closes #51263 by fixing forward

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

@mergify mergify Bot added the bug Something isn't working label Aug 7, 2026
@qyYue1389 qyYue1389 changed the title [MM][CG][BugFix] Fix Ernie-4.5-VL encoder CG postprocess for multi-pa… [MM][CG][BugFix] Fix Ernie-4.5-VL encoder CG postprocess for multi-path outputs Aug 7, 2026
Signed-off-by: qyYue1389 <yueqiuyang1389@gmail.com>

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

Thanks for the fix!

@khluu

khluu commented Aug 10, 2026

Copy link
Copy Markdown
Member

/ci run

@khluu
khluu enabled auto-merge (squash) August 10, 2026 21:46
@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #83216 for commit 0150aa45e907.

@github-actions github-actions Bot added the ready ONLY add when PR is ready to merge/full CI is needed label Aug 10, 2026
@khluu
khluu merged commit b2506d6 into vllm-project:main Aug 11, 2026
91 of 92 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working 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