search

Suzie1 / ComfyUI_Comfyroll_CustomNodes

Custom nodes for SDXL and SD1.5 including Multi-ControlNet, LoRA, Aspect Ratio, Process Switches, and many more nodes.
https://civitai.com/models/183551/comfyui-comfyroll-custom-nodes
588 stars 77 forks source link

multiscontrolnet error when using only 1 out of 3 input image slots #12

Closed MokkaBoss1 closed 1 year ago

MokkaBoss1 commented 1 year ago

Hi, I think I maybe have found another bug when using the multicontrol net stacker. If you leave any image input spaces (e.g. image 2 and / or image 3) empty, you get an error

"Error occurred when executing CR Multi-ControlNet Stack:

Comfyroll_ControlNetStack.controlnet_stacker() missing 2 required positional arguments: 'image_2' and 'image_3'

File "C:\sdxl\ComfyUI_windows_portable_nvidia_cu118_or_cpu\ComfyUI_windows_portable\ComfyUI\execution.py", line 151, in recursive_execute output_data, output_ui = get_output_data(obj, input_data_all) File "C:\sdxl\ComfyUI_windows_portable_nvidia_cu118_or_cpu\ComfyUI_windows_portable\ComfyUI\execution.py", line 81, in get_output_data return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True) File "C:\sdxl\ComfyUI_windows_portable_nvidia_cu118_or_cpu\ComfyUI_windows_portable\ComfyUI\execution.py", line 74, in map_node_over_list results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))"

json file attached (renamed as .txt) multicontrolnet stacker error.txt

Suzie1 commented 1 year ago

You need to connect an image to all three image inputs even if you are not using them. You can leave the controlnets for image2 and image3 Off, but they must have an input image. You can connect the same image to all three inputs. Suzie

MokkaBoss1 commented 1 year ago

ok thanks - it's clear

Mokkaboss1

Suzie1 commented 1 year ago

I'll add something to cover this in the Troubleshooting guide and also on the main post. Thanks for reporting it.