ComfyUI icon indicating copy to clipboard operation
ComfyUI copied to clipboard

Converted widgets -> inputs result in bad node when copy + paste.

Open rgthree opened this issue 2 years ago • 1 comments

We used to be able to copy + paste a node with widgets that were converted to inputs. At some point today, doing so now leaves the inputs, but puts the widgets back AND re-converting them to inputs duplicates the input.

See below:

bad_copy_paste

cc: @pythongosssss

rgthree avatar Oct 07 '23 03:10 rgthree

Looks like whatever broke this also broke all workflows currently using most fields in rgthree-comfy context nodes (now old workflows load with things like SEED/CFG, etc. no longer a forcedInput, and then overriding the values users want from the inputs).

EDIT: Just in case someone tries to reproduce with Context nodes, rgthree-comfy/commit/cc23fe1 divorces Context node UI impl from ComfyNode to fix the recent issues (and, thus, is fixed forward and not reproducible).

While this comment is resolved because of it, the original issue with KSampler, and I imagine a lot of other nodes, still exists.

rgthree avatar Oct 07 '23 03:10 rgthree