Acly / comfyui-tooling-nodes

Nodes for using ComfyUI as a backend for external tools. Send and receive images directly without filesystem upload/download.
GNU General Public License v3.0
319 stars 38 forks source link

Krita Parameter nodes with a subtype reset to defaults on workflow load #29

Open mmuellerdm opened 1 week ago

mmuellerdm commented 1 week ago

Edit: Note that this is on a fairly complex external ComfyUI installation with a large amount of custom nodes/plugins and it may well be one of those causing this. So if it can't be easily replicated, disregard.

When I reopen/load a workflow containing a Parameter node directly connected to a text input, it always resets itself to type "text", unless it is directly connected to a "CLIP Text Encode (Prompt)" node - in which case it always resets itself to type "prompt (positive)". I believe this means the type auto-detection code runs at each load and overwrites the user choice.

A side effect to this is that I have not found a way to get a node of type "prompt (negative)" to persist at all. They work as long as the workflow is open, but can't be saved.

There seems to be some screwiness with autodetection of number types as well (you can change the subtype but it will reset, and "number" really seems to mean "float" since it only connects with float inputs), but these at least reset to something useful once you get them set up.