LucianoCirino/efficiency-nodes-comfyui

Changes to aesthetic scores don't trigger new generation with KSampler Advanced until ComfyUI restart

alessandroperilli opened this issue · 3 comments

I'm not 100% sure this is about the KSampler Advanced node, but I'll flag it anyway.

In the following situation, with a fixed seed, a change in the aesthetic score values entering the CLIPTextEncodeSDXLRefiner node, won't push the KSampler Advanced node to generate a new refined image. It will simply regenerate the same image, as if no parameter was changed.

Screenshot 2023-10-26 at 10 23 55

To force the generation of an image that takes into account the modified aesthetic score values (all other things equal), I have to interrupt ComfyUI and restart it. As if something gets cached when it shouldn't*.

*I have a similar problem when I change the VAE in the Efficient Loader node: if I change the VAE setting from baked VAE to a specific VAE file, that is taken into account during the new generation. However, if I revert back to backed VAE for the subsequent generation, ComfyUI still tells me that it's using the specific VAE file. I have to interrupt ComfyUI and restart it to see the correct VAE setting being picked up.

It feels like something is aggressively cached when it shouldn't.

I have a similar issue with the Efficient Ksampler, with fixed seed, when i change a control nets' strength it doesn't update the image. With the default Comfui Ksampler it does.

Also experiencing this issue where Controlnet Stack strength change does not take effect until ComfyUI restarts when using Efficient KSampler (SDXL).

I also have the issue with adv controlnet apply.