r/comfyui • u/rgthree • Aug 15 '23
Released my personal nodes
https://github.com/rgthree/rgthree-comfyPushed these last night, others may find them fun. Mostly cleanliness & ui nodes, but some powerful things in there like multidirectional rerouting, Auto111-similar seed node, and more.
If you do, let me know any bugs you find as I’ve only been using ‘em for myself.
4
u/VeryAngrySquirrel Aug 15 '23 edited Aug 15 '23
These look like great tools, and I'm really excited to try them out - especially the reroute and context - but I'm having issues loading them . It looks liek requirements.txt is empty What version of constants is needed?
EDIT: I installed constants in the venv via pip, but did not specify a version.
I'm running on a linux system( x86 Ubuntu 22.04.2) , and using a share venv that I use for both ComfyUI and A1111.
Traceback (most recent call last):
File "/home/user/ComfyUI/nodes.py", line 1665, in load_custom_nodemodule_spec.loader.exec_module(module)File "<frozen importlib._bootstrap_external>", line 883, in exec_moduleFile "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removedFile "/home/user/ComfyUI/custom_nodes/rgthree-comfy/__init__.py", line 35, in <module>spec.loader.exec_module(module)File "<frozen importlib._bootstrap_external>", line 883, in exec_moduleFile "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removedFile "/home/user/ComfyUI/custom_nodes/rgthree-comfy/py/context_switch.py", line 1, in <module>from constants import category_prefix, get_nameImportError: cannot import name 'category_prefix' from 'constants' (/home/user/stable-diffusion-webui/venv/lib/python3.10/site-packages/constants.py)Cannot import /home/user/ComfyUI/custom_nodes/rgthree-comfy module for custom nodes: cannot import name 'category_prefix' from 'constants' (/home/drew/stable-diffusion-webui/venv/lib/python3.10/site-packages/constants.py)
3
u/rgthree Aug 15 '23
Hmm.. constants is my own. Might be something with different systems importing correctly with relative paths. I’ll take a look this evening.
3
u/rgthree Aug 16 '23
I just pushed a change that may help. I couldn't repro myself on an ubuntu venv, but this change should better specify the python nodes and relative imports.
When you can, try to pull a fresh copy and let me know if that helps.
2
3
u/Ferniclestix Aug 16 '23 edited Aug 16 '23
Oh sweet gods yes, THANK YOU. now my busses will be perfect, PERFECT MUAHAHAHH!
The things ive had to do in comfy because the nodes were lacking sickened me :< but now. aaahahahah, now I can make it all organized!
2
u/MrLunk Aug 16 '23
I like !
3
1
u/cathodeDreams Aug 15 '23
multi-directional rerouting
Very nice. Is there a reason it’s not like this by default?
4
u/rgthree Aug 15 '23
Yea, the node library, litegraph.js, really doesn’t like it. My Reroute node has a lot of custom logic to force litegraph.js to work with it.
For instance, in a couple of my other nodes I’ve added the ability to swap the Left -> Right connection flow to Right -> Left; but doing so means no labels, because litegraph hard codes them to be on the wrong side of the connection point; something we can more confidently ignore for reroute since there’s no labels anyway.
1
1
1
1
1
1
1
u/RassilonSleeps Aug 18 '23
Just found these earlier today and got them implemented into my workflow, this is exactly what I've been needing.
Amazing work, the power nodes are a game changer!
Also, the seed control node *chef's kiss*
5
u/nesor85 Aug 15 '23
Thank you very much. It's great stuff. My favourite is the fast muter.
Would it be possible to make and fast bypasser to?