Skip to content

[Fix] diffusion: Drop stale mismatch_msg from FLUX 3 video VAE gate call - #45778

Merged
hnyls2002 merged 1 commit into
mainfrom
alexnails/flux3-vae-gate-kwarg
Oct 22, 2026
Merged

hnyls2002 merged 1 commit into
mainfrom
alexnails/flux3-vae-gate-kwarg

Conversation

@alexnails

@alexnails alexnails commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Motivation

#41894 calls BitExactFusionGate.accept_or_fallback(..., mismatch_msg=...), but #44690 removed that keyword from the gate and every other caller. The first fused QKV norm+RoPE call in the FLUX 3 video VAE raises TypeError: unexpected keyword argument 'mismatch_msg'. This is the remaining break after #45775.

Modifications

Drop the keyword.

Tests

test/registered/kernels/ops/diffusion/test_flux3_video_qknorm_rope.py on H200: test_natten_prepare_qkv_matches_eager_and_reuses_rope_cache fails with the TypeError with #45775 alone; all 4 pass with this change.

🤖 Generated with Claude Code


CI States

Latest PR Test (Base): ❌ Run #39748250854
Latest PR Test (Extra): ❌ Run #39748250779
Latest PR Test (AMD ROCm 10): ❌ Run #39748250778

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added the diffusion SGLang Diffusion label Oct 22, 2026
@hnyls2002
hnyls2002 merged commit d0a7565 into main Oct 22, 2026
114 of 125 checks passed
@hnyls2002
hnyls2002 deleted the alexnails/flux3-vae-gate-kwarg branch October 22, 2026 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

diffusion SGLang Diffusion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants