ComfyUI: Remove --normalvram#1644
Conversation
Removed 'normalvram' option
There was a problem hiding this comment.
Code Review
This pull request removes the --normalvram argument from ComfyUI's configuration options and initial value selection logic. Feedback identifies a critical backward compatibility issue: removing the option from the UI does not clear it from existing user configurations, which will cause ComfyUI to crash upon encountering the now-unknown argument. It is recommended to filter this flag out during the package execution process. Additionally, the GPU detection logic is susceptible to an InvalidOperationException if no GPUs are found, which should be mitigated by providing a default value before calling Max().
|
Added stripping of existing --normalvram argument from settings.json upon settings load and atomic rebuild. Launch argument removed from comfy configuration and covers users with pre-existing set arg from either manually set, or when SM set it automatically for 4-8GB GPU during install. |
--normalvram has been removed from ComfyUI as of this PR and will throw unknown argument error at package launch if the user has it enabled in Launch Options window as it appears there.
It's function is implied by default by ComfyUI unless it is overridden by the user or by certain hardware checks by Comfy's internal logic.
Since SM sets normalvram automatically on installs with 4-8GB vram GPUs, any user before this change that has their comfy install using the repo commits instead of releases, their comfy launch will hit a launch argument error.