rgthree-comfy
rgthree-comfy copied to clipboard
Feature Requests:Enable fast bypass/mute node support to convert to Group nodes
Currently fast bypass/mute nodes disappear when converted to group nodes, is it possible to make bypass/mute support conversion to group nodes? https://github.com/comfyanonymous/ComfyUI/pull/1776 One way I can think of that might work, make a simple version of the fast bypass/mute node that has only one input and one widget, just as simple as a normal node, and that might be able to be converted to a group node without any problems.
You mean when Fast mute/bypass nodes are selected to be grouped? Or when one of their inputs are put in a group?
Looking through the notes on these group nodes, I think this is going to be the problem:
Frontend extension nodes that need to interact with the node won't work propertly with this, as they are only created when the prompt is queued and never actually added to the graph.
The Fast Muter/Bypasser nodes are exactly that; they're front end extension nodes that mute/bypass nodes directly. So these new group nodes seem to make it not possible to have the Fast Muter/Byapsser nodes interact with nodes inside the group node (because they don't existing in the graph until it's sent to the server).
It does look like Fast Muter/Bypasser nodes work as expected with the actual group node itself.
@rgthree
After I converted some nodes including fast bypass/mute
to group nodes, fast bypass/mute
would not show up on the group nodes.
So the current fast bypass/mute
doesn't support conversion to group nodes, and I thought it would be useful to make a simple version of fast bypass/mute
that can convert to group nodes!
This is another case.
Even if fast bypass/mute
connects to a node that is not included in a group node, fast bypass/mute
will not convert properly!