shiimizu / ComfyUI-PhotoMaker-Plus

PhotoMaker for ComfyUI
GNU General Public License v3.0
149 stars 14 forks source link

PhotoMaker node conflict #19

Closed NicholasKao1029 closed 5 months ago

NicholasKao1029 commented 5 months ago

image

With the new Photomaker nodes in ComfyUI how would you reccomend users deal with the conflict?

ParticleDog commented 5 months ago

conflicts +1

cvar66 commented 5 months ago

Latest ComfyUI update broke my workflow

nm, just had to uninstall and reinstall under the new plus name

cvar66 commented 5 months ago

OK it seems it's not working properly afterall. No errors but the images are different on not as good. Going to try and rollback Comfy.... rolling back didn't help. I regret updating comfy.

Just realized there is another new node called PhotoMaker Encode Plus. Just replaced the old one with this and its working again.

shiimizu commented 5 months ago

You can ignore the conflict warning. That's a ComfyUI Manager bug that doesn't check for namespaces because the actual node being used is the one in ComfyUI. The loader node in this repo is identical to the one in ComfyUI and it exists for previous versions of ComfyUI that don't have it. The images look different because of the reasons listed in the readme.

EDIT: This has been fixed in ComfyUI Manager: