Skip to content
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

When bypassing a a group node containing a Apply Advanced ControlNet 🛂🅐🅒🅝, the positive and negative conditioning get switched. #70

Open
dchatel opened this issue Feb 14, 2024 · 3 comments

Comments

@dchatel
Copy link

dchatel commented Feb 14, 2024

This is quite simple. The steps to reproduce simply consist in creating a group node with Apply Advanced ControlNet 🛂🅐🅒🅝, then bypassing that group node.
When bypassed, the positive and negative conditioning will switch places: the positive get linked to the negative and vice versa. which leads to... pretty creepy but sometimes hilarious stuff.

This is what we get when bypassing Apply Advanced ControlNet 🛂🅐🅒🅝 when ungrouped, so this is also what is expected when bypassing a grouped node: https://ibb.co/WWw3ZBT
But this is what we get when bypassing a grouped node containing Apply Advanced ControlNet 🛂🅐🅒🅝 (Warning: it hurts the eyes a little bit): https://ibb.co/x569F61

@Kosinkadink
Copy link
Owner

I have no custom javascript at all in my node, so my hunch is this may be a bug with how ComfyUI handles bypassing grouped nodes. Can you see if the issue also happens if you use the vanilla Apply ControlNet (Advanced) node that's built into comfy?

@dchatel
Copy link
Author

dchatel commented Feb 16, 2024

Yes, it happens even with the vanilla apply controlnet (advanced). I've forgot to update you on the matter, my bad.

@Kosinkadink
Copy link
Owner

Gotcha, you should open up an issue on ComfyUI repo - there is nothing I can do about this on my end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants