comfyanonymous / ComfyUI

The most powerful and modular diffusion model GUI, api and backend with a graph/nodes interface.
https://www.comfy.org/
GNU General Public License v3.0
51.13k stars 5.38k forks source link

Suggestion: Red warning status when the node is not installed, you can keep the parameters #3655

Open rui40000 opened 3 months ago

rui40000 commented 3 months ago

red

When a node is not installed it displays a red alert and none of the parameters are displayed in that state, although reimporting workflow after installation solves the problem. There are also some special cases, such as the user can not install the original node, to use the same type of node replacement, he can not get the original parameters. As well as in some web platform online display can not browse the parameters (Openart).

Is it possible to keep the display of the parameters in the red alert state? It seems that this also does not is no problem :)

ltdrdata commented 3 months ago

In the current ComfyUI structure, parameters cannot be accurately represented for nodes that have not been loaded. A structural change to the workflow is needed.

shawnington commented 3 months ago

In the current ComfyUI structure, parameters cannot be accurately represented for nodes that have not been loaded. A structural change to the workflow is needed.

lots of changes are needed, like, the server work flow format not being the same as the workflow.json, and there being no translation layer. Silly decisions were made at critical junctures.

Your nodes usually report errors first though, I officially nominate you for elevation to maintainer.