Fix UnboundLocalError: multitalk_audio_stride referenced before assignment in WanVideoSampler#2032
Open
kylinjg wants to merge 1 commit into
Open
Fix UnboundLocalError: multitalk_audio_stride referenced before assignment in WanVideoSampler#2032kylinjg wants to merge 1 commit into
kylinjg wants to merge 1 commit into
Conversation
…nment multitalk_audio_stride is only assigned inside the `if multitalk_embeds is not None` block, but is read unconditionally later in WanVideoSampler.process. Any sampler run without multitalk_embeds (standard T2V/I2V, VACE, Fun-Control, etc.) raises: UnboundLocalError: cannot access local variable 'multitalk_audio_stride' Initialize it to None alongside the other multitalk locals. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
WanVideoSamplerraises anUnboundLocalErroron any run that does not passmultitalk_embeds(i.e. the large majority of workflows — standard T2V/I2V, VACE, Fun-Control, etc.):Root cause
multitalk_audio_strideis only bound inside theif multitalk_embeds is not None:branch:but it is later read unconditionally:
When
multitalk_embedsisNone, the name is never assigned, so the later read throws.Fix
Initialize
multitalk_audio_stride = Nonealongside the other MultiTalk locals (multitalk_audio_embeds,audio_emb_slice,audio_features_in), so the unconditional read is always safe. One line, no behavior change for the MultiTalk path.Reproduction
Run any VACE / T2V / I2V workflow through
WanVideoSamplerwithout a MultiTalk embed connected → error before this patch, works after.