WASasquatch / was-node-suite-comfyui

An extensive node suite for ComfyUI with over 210 new nodes
MIT License
1.24k stars 178 forks source link

No module named 'models.blip' #210

Open sanshaoyeyang opened 1 year ago

sanshaoyeyang commented 1 year ago

cant run the blip loader node!please help

!!! Exception during processing !!! Traceback (most recent call last): File "D:\AI\ComfyUI_windows_portable\ComfyUI\execution.py", line 152, in recursive_execute output_data, output_ui = get_output_data(obj, input_data_all) File "D:\AI\ComfyUI_windows_portable\ComfyUI\execution.py", line 82, in get_output_data return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True) File "D:\AI\ComfyUI_windows_portable\ComfyUI\execution.py", line 75, in map_node_over_list results.append(getattr(obj, func)(**slice_dict(input_data_all, i))) File "D:\AI\ComfyUI_windows_portable\ComfyUI\custom_nodes\was-node-suite-comfyui\WAS_Node_Suite.py", line 10581, in blip_model from models.blip import blip_decoder ModuleNotFoundError: No module named 'models.blip'

WASasquatch commented 1 year ago

Are you using the BLIP Model Loader? I think I forgot to append the repo path. https://github.com/WASasquatch/was-node-suite-comfyui/commit/573464e4ff27b88a0a48f7a5d977b47672d01c97

sanshaoyeyang commented 1 year ago

您使用的是 BLIP 模型加载器吗?我想我忘记附加存储库路径。573464e

Sorry for not logging in and replying to you in a timely manner in the past two days. Thank you for your reply. Yes, I encountered the above issue while using the BLIP model loader you mentioned. After adding the code according to your reply, the result remains the same and did not solve the problem. I found an error when installing requirements.txt in the BLIP folder. I suspect this issue may be the cause, but I am unable to resolve it.

image
WASasquatch commented 1 year ago

I am not familiar with tokenizers install or TOML installations, I may not be of much help but I will look into it as soon as possible.

On Tue, Sep 19, 2023, 3:42 PM sanshaoyeyang @.***> wrote:

您使用的是 BLIP 模型加载器吗?我想我忘记附加存储库路径。573464e https://github.com/WASasquatch/was-node-suite-comfyui/commit/573464e4ff27b88a0a48f7a5d977b47672d01c97

Sorry for not logging in and replying to you in a timely manner in the past two days. Thank you for your reply. Yes, I encountered the above issue while using the BLIP model loader you mentioned. After adding the code according to your reply, the result remains the same and did not solve the problem. I found an error when installing requirements.txt in the BLIP folder. I suspect this issue may be the cause, but I am unable to resolve it. [image: image] https://user-images.githubusercontent.com/30750058/269112068-45fa0eda-9833-4414-bad6-5ef37cb2d551.png

— Reply to this email directly, view it on GitHub https://github.com/WASasquatch/was-node-suite-comfyui/issues/210#issuecomment-1726635370, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIZEZPZICNUJ2AJOLEGYATX3INU7ANCNFSM6AAAAAA43OYAGI . You are receiving this because you commented.Message ID: @.***>

liangkun43 commented 1 year ago

After I installed the "Comfyui Lama" node, my latest WAS node also received the same error. Another node, "Comfy Clip Blip Node" also received the same error, but when I used this version, the problem was resolved.WAS node still received this error 微信截图_20231005004630

WASasquatch commented 1 year ago

Probably a fair bet you should not have two nodes trying to import the same namespaces if you don't want conflicts. For example, that Comfy Clip Blip says you need transformers 4.26.1 like mine, but instead, installs 4.15 (transformers==4.15.0)

liangkun43 commented 1 year ago

I uninstalled "Comfy Clip Blip Node" and transformers, and reinstalled WAS node, but still received this error. 微信截图_20231005113633 微信截图_20231005113659

WASasquatch commented 1 year ago

Does the repo even exist under was-node-suite-comfyui/repos/BLIP? All it's doing is appending the path and importing as normal. So any issue with would be confliction or just not there.

On Thu, Oct 5, 2023, 7:33 AM liangkun43 @.***> wrote:

I uninstalled "Comfy Clip Blip Node" and transformers, and reinstalled WAS node, but still received this error. [image: 微信截图_20231005113633] https://user-images.githubusercontent.com/135417715/272938307-357dcdbf-2f84-49ce-8e2e-93f35e27eade.png [image: 微信截图_20231005113659] https://user-images.githubusercontent.com/135417715/272938317-83e23c06-d29a-404b-9db6-473e568905ef.png

— Reply to this email directly, view it on GitHub https://github.com/WASasquatch/was-node-suite-comfyui/issues/210#issuecomment-1749033618, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIZEZOJ5GCWMFLHVPYKUUTX53AMFAVCNFSM6AAAAAA43OYAGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBZGAZTGNRRHA . You are receiving this because you commented.Message ID: @.***>

liangkun43 commented 1 year ago

Is it related to this issue?

WASasquatch commented 1 year ago

Is it related to this issue?

No, that's just a PR to change things, which would likely also break BLIP because BLIP must be imported relative, as it itself uses local imports. So if you just import it from anywhere it can't import it's resources and will error out, cause it's not part of its package namespace

WASasquatch commented 1 year ago

Alright so BLIP is now a shipped module of WAS-NS. It doesn't need to download the repo or use it with sys.path.append