Conversation
…e prompt The R2V demo prompt referenced <Audio 1> but no audio input is connected (ref_audios.ref_audio_0 is None). This is the same dangling-tag issue fixed in Comfy-Org#1129; the reference silently no-ops when users run the template as-is. Restore the prompt to reference only the connected images.
🔤 Spellcheck Results
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan includes up to 4 reviews per rolling hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe Minimax video workflow prompt no longer requires ChangesMinimax video workflow
Suggested reviewers: Merge Risk: ⚪ Minimal · up to The template now references only connected image inputs and keeps its manifest synchronized. No actionable merge-blocking risk remains after normal validation. 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The R2V demo prompt was updated in #1137 to say "Use <Picture 2> and <Picture 1> as reference frames and <Audio 1> exactly as it is.", but no audio input is connected:
ref_audios.ref_audio_0has link None and the workflow has no LoadAudio node.This is the same dangling-tag issue fixed in #1129 (docs#1386): the
<Audio 1>reference silently no-ops when users run the template as-is.Fix: restore the prompt to reference only the connected images. Syncs manifest accordingly.
Ref: Comfy-Org/docs#1447 (docs page documents the R2V tag system)