-
Notifications
You must be signed in to change notification settings - Fork 6.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request] Chip based groups / general group update #497
Comments
nice idea! |
Yes, wanted to open similar request myself, not sure of why reroute is even needed, it could just figure what is ingoing and outgoing into group. And just create minimized node. Just inputs/outputs need ability to have names/be renamed (I guess maybe thats why you added them?). In any case ability to save such groups as "user made nodes" and potentially ability to easily share them would be gold. I also think blender has such feature of converting groups to nodes, which you can then "step into" and edit if needed. |
In this case the reroutes are just a way of keeping things neat - and having pass-through flow. (So if i was to use latent upscaling after the txt2img - i can pull all the required elements from the end of the txt2img group without having tonnes of wires coming from the initial nodes. Which also allows for more modular plug'n'play node groups. But yes, ideally if this was to be implemented the inputs and outputs of the node group would be set inside the group element rather than using reroutes. I agree with the renaming too - currently even the reroutes only show the type of data rather than the set title. |
The litegraph used in ComfyUI already has this functionality and it's not bad. Perhaps should just implement it? |
what really , didnt think this supported in litegraph , will try look into it |
You might want to read through/follow these other related issues, as a bunch of stuff seems to have already been tried/explored/etc in this space:
Also wanted to include this idea/note I posted on the other issue here; RE: litegraph bugs/limitations and a potentially better alternative:
|
I think it would be useful to have a new ChipNode Group with a custom amount of IO 'reroute' nodes, and a customizable display based on the contents of the group. Being able to build new 'chips' within the UI would be quite powerful, and would help clean up routing, as well as simplifying re-useable groups in a way that Node Templates doesn't quite reach in its current form.
As an example a txt2img chip could look something like this.
Being able to expand and collapse to edit the contents would also be helpful, although the group hierarchy issue (#477 ) would probably need to be fixed as well.
Having a way to lock the contents of the group, so that if it expands over other nodes it doesn't automatically include them into the group, which is a slight issue with the current implementation would also be good. Auto selecting the groups contents when you select a group too.
For another reference in regards to ChipNodes - similar functionality to the following logic gate app is essentially what I mean. https://github.com/SebLague/Digital-Logic-Sim
The text was updated successfully, but these errors were encountered: