docs: WanImageToVideo FPS, ComfyUI Manager update, memory offload defaults (consolidated)#1272
Open
lin-bot23 wants to merge 4 commits into
Open
Conversation
Users commonly ask what frame rate the WanImageToVideo node outputs at. The length parameter gives frame count but FPS was undocumented. Wan video generation uses a fixed 16 FPS, documented in tutorial workflows but missing from the node reference docs. Source: Mintlify AI Assistant analytics data (2026-07-15)
Users asked about updating ComfyUI Manager, which is a separate custom node from ComfyUI core. The update page covered ComfyUI itself but did not mention how to update Manager. Adds three methods: via Manager UI, via Git pull, and notes for the Portable version. Source: Mintlify AI Assistant analytics data (2026-07-15)
Users commonly ask whether memory offloading is on by default. The flag table listed defaults per-flag but there was no summary of the combined default behavior. This adds a sentence explaining that async weight offloading and dynamic VRAM are enabled by default on Nvidia GPUs. Source: Mintlify AI Assistant analytics data (2026-07-15)
lin-bot23
changed the title
This was referenced Jul 15, 2026
…07-15-memory-offload-defaults
📝 WalkthroughWalkthroughDocumentation now records Nvidia defaults for VRAM flags and adds ComfyUI Manager update instructions for UI, Git, and portable Windows installations, including the ChangesStartup flag defaults
ComfyUI Manager updates
Suggested reviewers: 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches🧪 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@installation/update_comfyui.mdx`:
- Line 136: Update the opening ComfyUI Manager description in the installation
documentation to state that its separate custom-node update workflow applies to
manual installations, while Portable installations bundle Manager with ComfyUI
and update it through the core process. Ensure the corresponding wording at the
additionally affected Manager description remains consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 5de15728-e17d-4a90-97e9-db4045c8e849
📒 Files selected for processing (2)
development/comfyui-server/startup-flags.mdxinstallation/update_comfyui.mdx
|
|
||
| ## How to Update ComfyUI Manager | ||
|
|
||
| ComfyUI Manager is a custom node that provides installation and management of other custom nodes, models, and snapshots. It is separate from the ComfyUI core code and has its own update process. |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Clarify that Manager’s update model depends on the installation type.
Line 136 says Manager is separate from ComfyUI core and has its own update process, but the Portable section says it is bundled into core and updated with ComfyUI. Make the opening description explicitly scope the separate custom-node workflow to manual installations. Otherwise the docs tell a split-brain update story.
Suggested wording
-ComfyUI Manager is a custom node that provides installation and management of other custom nodes, models, and snapshots. It is separate from the ComfyUI core code and has its own update process.
+ComfyUI Manager provides installation and management of other custom nodes, models, and snapshots. Its update process depends on the installation type: manual installations manage it separately, while Windows Portable bundles it into ComfyUI core.Also applies to: 159-160
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@installation/update_comfyui.mdx` at line 136, Update the opening ComfyUI
Manager description in the installation documentation to state that its separate
custom-node update workflow applies to manual installations, while Portable
installations bundle Manager with ComfyUI and update it through the core
process. Ensure the corresponding wording at the additionally affected Manager
description remains consistent.
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
1 participant
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.
Summary
Consolidated PR merging 3 related changes from the 2026-07-15 analytics batch that had overlapping file changes:
Changes
WanImageToVideo output frame rate (
built-in-nodes/WanImageToVideo.mdx) — Added note confirming Wan video uses fixed 16 FPS with duration calculation example.ComfyUI Manager update instructions (
installation/update_comfyui.mdx) — Added section covering Manager UI, Git, and Portable version update methods.Memory offloading default behavior (
development/comfyui-server/startup-flags.mdx) — Added summary that async weight offloading and dynamic VRAM are enabled by default on Nvidia GPUs.Supersedes
Checklist