search

cubiq / ComfyUI_InstantID

Apache License 2.0
945 stars 50 forks source link

"Doesn't use diffusers" but then error on basic workflow that was only solved by adding the diffusers safetensors model #168

Closed ewebgh33 closed 3 weeks ago

ewebgh33 commented 3 weeks ago

As the title says I am confused Do we need the safetensors model or not? Loaded the basic workflow It failed, said I was missing the model Added the diffusers safetensors model, it works So It DOES use diffusers safetensors? What am I missing here, the readme says right at the top, doesn't use diffusers, is different.

cubiq commented 3 weeks ago

safetensors is the model. diffusers is a library.

ewebgh33 commented 3 weeks ago

Maybe I'm confused because the model's full filename is diffusion_pytorch_model.safetensors

Maybe this is a language thing I mean, I solved my problem by installing that model so, cool But The readme says "This extension differs from the many already available as it doesn't use diffusers but instead implements InstantID natively and it fully integrates with ComfyUI." But then you have to install a safetensors file whose filename is literally "diffusers" Sorry for the confusion?

cubiq commented 3 weeks ago

the filename literally says diffusion not diffusers . Diffusers is library not a file. Ask chatgpt I'm sure it's able to explain it very quickly

ewebgh33 commented 3 weeks ago

Ah OK got it, got it. :) It is a language thing Many words, many languages in my head Thank you