Kosinkadink / ComfyUI-Advanced-ControlNet

ControlNet scheduling and masking nodes with sliding context support
GNU General Public License v3.0
622 stars 60 forks source link

[ Feature Request ] Requesting the Multi-Controlnet feature for use with AnimateDiff Comfyui #7

Closed Ketorami closed 1 year ago

Ketorami commented 1 year ago

I noticed that AnimateDiff for ComfyUI works well when using just a single Controlnet. However, when I tried using other nodes to combine multiple Controlnets, I couldn't get the AnimateDiff node to work. It's really disappointing that I can only use it with a single Controlnet. So, I would like to ask if you have any plans to develop a Multi-Controlnet Node for ComfyUI AnimateDiff. It's truly unfortunate that Multi-Controlnet doesn't seem to work for AnimateDiff.

Kosinkadink commented 1 year ago

Can you send me an example of one of your workflows? People have been using like half a dozen controlnets at a time with Kosinkadink/ComfyUI-AnimateDiff-Evolved without issue so far. You don't need to combine controlnets with separate nodes to combine them, the typical usage is chaining them together.

Ketorami commented 1 year ago

The problem is that I cannot find the node to combine Controlnet together, and I don't know how to combine multiple Controlnet nodes to create a single image like A1111, using features such as "Depth," "softedge," and "Openopse."

I'm using ComfyUI_Comfyroll_CustomNodes to combine them because it seems to have a node for use Multi-Controlnet. However, it doesn't work.

AnimateDiff Complete contolnet2  Multi-Controlnet Fail

Kosinkadink commented 1 year ago

You can chain control nets together by plugging the positive and negative conditioning of one controlnet into the next. See if that works for you as a replacement for the multicontrolnet node. I'd share a quick screenshot, but I'm not home rn.

Ketorami commented 1 year ago

Holy, that's genius! Why couldn't I think of it?! I don't need to do anything complex at all. Just connect the Positives and Negatives of ControlNTE together, and then it will work! I'm so confused why I missed such a simple idea like this