From de1ad8512a82e7e7fb3dfcdade68399919e205c5 Mon Sep 17 00:00:00 2001 From: asagi4 <130366179+asagi4@users.noreply.github.com> Date: Sun, 15 Dec 2024 00:13:16 +0200 Subject: [PATCH] Note about cache problem --- README.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index 90427e6..cdb8169 100644 --- a/README.md +++ b/README.md @@ -165,4 +165,5 @@ The parameters affect how the masked and unmasked prompts are combined to produc # Known issues -- For some reason, ComfyUI isn't caching the dynamic graphs after all. :( I'm sure it did when I tested previously. +- ComfyUI's caching mechanism has an issue that makes it unnecessarily invalidate caches for certain inputs; you'll still get some benefit from the lazy nodes, but changing inputs that shouldn't affect downstream nodes (especially if using filtering) will still cause them to be recomputed because ComfyUI doesn't realize the inputs haven't changed. +I have a way to solve this, but it requires modifying ComfyUI code and I'm not sure that it doesn't break something else.