-
Notifications
You must be signed in to change notification settings - Fork 14
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
I THINK I fixed the nodes losing connection issue, also another issue with mxSeed #25
Comments
Yes, it's all due to the int-float switch. There have been issues with this before. Fortunately, the new version of ComfyUI offers more options to help prevent such errors. So, for now, I don't plan to split the nodes, even though it's the easiest and most reliable solution. |
Glad to hear! I'm making a bunch of minimalist sliders and bool buttons, aim is to take as little screen space as possible, all based on your nodes, do you mind if I put them up to share? You will of course be credited and your GitHub linked. Oh btw I've been trying to change the node back ground color, but can only make it work in this.node.onAdded, is there a work around? Thanks and...CHEERS! |
My nodes are distributed under the GPLv3 license, so there are no issues with freely using my code in other projects. |
Hi, sorry to bother you again! I was trying to migrate my sliders code to your router code, by using |
I think the connection issue is caused by your dynamic switch between float and int? I couldn't find any actual errors so I rewrote your code and divided the sliders to INT and FLOAT, also added a fill bar and a bunch of customization options, it's at: https://github.com/lordwedggie/xcpNodes
Your mxSeed is awesome! I'm using it in all my workflows, however, upon loading a new workflow, the first time you click on the seed it doesn't create a new seed, but launches the queue on the same seed again? It's a minor issue but hopefully it can be fixed. Cheers!
The text was updated successfully, but these errors were encountered: