-
Notifications
You must be signed in to change notification settings - Fork 230
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
Error occurred when executing Zoe-DepthMapPreprocessor #2
Comments
I just tested on ComfyUI_windows_portable and couldn't reproduce that bug. It seems to be a MPS-specific bug like you said. |
Can you try to upgrade pytorch to the newest version? This bug come from the pytorch side so if that doesn't work then I will just force it to use the CPU if the device is MPS. |
I used to get a similar issue to the one in the OP, but uninstalling https://github.com/space-nuko/ComfyUI-Disco-Diffusion resolved it for me. Might check for any potential conflicts with other custom nodes (especially if you have this one). |
I think the source of this issue is that midas_repo import "midas" which seems to conflict with "midas" from other repos Line 4 in e6c9efb
|
Can you guys test the new commit? I moved midas_repo to different location and rename stuffs |
I think I actually can't retest this repo's
Currently on commit 2a0ee39 |
@Lightsockie I fixed #3. Can you try again? |
Up to date as of earlier this evening and receiving pretty much the same issue expected size to be one of int or Tuple[int] or Tuple[int, int] or Tuple[int, int, int], but got size with types [, ] File "F:\ComfyUI\execution.py", line 151, in recursive_execute pytorch is also up to date on the nightly channel, using cuda 12.1 and bf16 flag in comfy |
Hi, |
same issue as @KevinParnell |
Sorry about the delay, but confirming issue still exists. Only using this custom_node and ComfyUI-Manager Same error as OP.
jk I see the README now says not to do this |
Hi M1max64go here I get this : expected size to be one of int or Tuple[int] or Tuple[int, int] or Tuple[int, int, int], but got size with types [, ] File "/Users/gbkm1max/ComfyUI/execution.py", line 151, in recursive_execute |
Same error on Windows 11 with Python 3.11.6 and torch 2.1.0+cu121. |
this also happens on linux/cuda to fix this:
file /comfyui_controlnet_aux/src/custom_midas_repo/midas/backbones/beit.py replace line 47 with
|
Your fix worked for me. Thanks. |
it does worked in windows alse.can you merge it to the main branch. |
@cubiq : thank for the fix. No one willing to create a PR ? |
PR #71 pushed |
Error occurred when executing Zoe-DepthMapPreprocessor: The operator 'aten::upsample_bicubic2d.out' is not currently implemented for the MPS device. If you want this op to be added in priority during the prototype phase of this feature, please comment on pytorch/pytorch#77764. As a temporary fix, you can set the environment variable File "/Users/gbkm1max/ComfyUI/execution.py", line 152, in recursive_execute |
@kameleo333 Fixed at e3ac5ef |
Error occurred when executing MiDaS-DepthMapPreprocessor: Connection error, and we cannot find the requested files in the disk cache. Please try again or make sure your Internet connection is on. File "I:\Blender_ComfyUI\ComfyUI\execution.py", line 153, in recursive_execute Close |
add config path judge, change default annotator_ckpts_path consistent with utils.py
Running this on Mac OS with MPS support in nightly Pytorch.
I have no idea if this is related but noticed that in:
custom_nodes/comfyui_controlnet_aux/node_wrappers/zoe.py
it can't import
import controlnet_aux
More likely I'm thinking this must have something to do with the way torch is casting integers.
The text was updated successfully, but these errors were encountered: