search

vivax3794 / ComfyUI-Sub-Nodes

Finally sub nodes in comfy!
120 stars 4 forks source link

A small bug #18

Open 807502278 opened 1 month ago

807502278 commented 1 month ago

When the output of input is connected to something else and then to default, an error "2" will be reported when loading with subgraph.

For example, it was like this at the beginning: image Then I added a default-node, connected the first input to inp, and an error "Error loading input/outputs for subgraph:2" would be reported when loading this. image

Of course, it would be normal if the input was deleted and recreated and then connected to inp first. But if it was a very complex sub-node and the connection sequence was accidentally wrong, it would be very headache-inducing.

807502278 commented 1 month ago

It is just like when I connect input's "FLOAT.a.88" to inp, it will be automatically updated to "FLOAT.inp.**".

807502278 commented 1 month ago

Then there is another small problem. After I successfully loaded using the above example, the B input also became the default value.
image

807502278 commented 2 weeks ago

Hi @vivax3794 , when will you update? : )

vivax3794 commented 2 weeks ago

Hi @vivax3794 , when will you update? : )

Sorry for the lack of updates lately, just started uni so lifes been busy. Will try to knock out a few updates this week

807502278 commented 2 weeks ago

Great, you're finally back