jags111 / efficiency-nodes-comfyui

A collection of ComfyUI custom nodes.- Awesome smart way to work with nodes!
https://civitai.com/models/32342
GNU General Public License v3.0
907 stars 93 forks source link

Token interpretation inconsistency #80

Open LuisBurdallo opened 8 months ago

LuisBurdallo commented 8 months ago

I thought it should at least get it as close as possible but after some testing it I can't really match the exact A1111 generation. There must be more to it. I am attaching a couple of pictures and maybe you can tell me if I am doing anything wrong. I've tried mean, length etc... Also all the other conversion modes. Can you help with this huge problem which is not being able to fully migrate to ComfyUI as it feels pretty inconsistent and I got some beautiful images created that I wish I could port over.

Snippet_002

Snippet_003

jags111 commented 8 months ago

this issue you are referring relates to basic comfyUI sampler configuration and not in the eff scripts. so very little can be done to change same as the code in comfy works in a different way than auto1111. i have little control on same.

LuisBurdallo commented 8 months ago

Yes I know it is an issue by design and how ComfyUI handles weights. I just thought that the A1111 interpretation would yield the same results but I guess that is not gonna happen ever as it's developer seems to be totally against it.

jags111 commented 7 months ago

looking into it and need a few days to resolve same