ModelSurge / sd-webui-comfyui

An extension to integrate ComfyUI workflows into the Webui's pipeline
MIT License
516 stars 32 forks source link

ConfyUI-Extension installation outdated #227

Open schoenid opened 3 months ago

schoenid commented 3 months ago

Trying to install some of the newer nodes causes venv to be destroyed.

See also https://github.com/WASasquatch/was-node-suite-comfyui/issues/457#issuecomment-2282901221

I had a coomment there:

Ah you are using the extension. That is a pretty old extension which hasn't had an update in 7mo.

PladsElsker commented 3 months ago

It is old and pretty much no longer maintained. That being said, there is an option in the settings to select separated venvs for A1111 and ComfyUI to work together nicely.

PladsElsker commented 3 months ago

117

PladsElsker commented 3 months ago

Ah, maybe there isn't one? I don't remember

PladsElsker commented 3 months ago

https://github.com/ModelSurge/sd-webui-comfyui/blob/10e95a42e27f7e7c2b15284491d1e09f2ea9a971/lib_comfyui/comfyui_process.py#L51-L71

If you install ComfyUI separately with its own venv, and reference the install location in the settings, it should work, assuming the extension still works at all