You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is too much cloned defaults in widgetDefaults.ts, but it always will be not enough for endless custom nodes for ComfyUI.
And if we change widgetNames to another (not from our node widget) - this will break correct widget loading in UI (tried with KSampler Config (rgthree), which has field "steps_total", if widgetNames is just "steps" - widget will show default value instead of real node value. If
widgetNames is "steps_total" - there is no default/step/min/max values.)
Better solution will be using only widgetIdxs, remove all widgetNames and change it to widgetTypes (e.g. steps, cfg, etc...), which can be used for any selected widgetIdxs.
Also, sometimes may be need to set custom widget parameters instead of its type (min/max/step).
The text was updated successfully, but these errors were encountered:
There is too much cloned defaults in widgetDefaults.ts, but it always will be not enough for endless custom nodes for ComfyUI.
And if we change widgetNames to another (not from our node widget) - this will break correct widget loading in UI (tried with KSampler Config (rgthree), which has field "steps_total", if widgetNames is just "steps" - widget will show default value instead of real node value. If
widgetNames is "steps_total" - there is no default/step/min/max values.)
Better solution will be using only widgetIdxs, remove all widgetNames and change it to widgetTypes (e.g. steps, cfg, etc...), which can be used for any selected widgetIdxs.
Also, sometimes may be need to set custom widget parameters instead of its type (min/max/step).
The text was updated successfully, but these errors were encountered: