diff --git a/README.md b/README.md index ae83c09..af7eea4 100644 --- a/README.md +++ b/README.md @@ -109,12 +109,17 @@ The other thing worth trying is clearing out all the custom node javascript from - remove everything there EXCEPT for `core`. Leave `core` (it's ComfyUI stuff) - restart Comfy (all custom nodes will reinstall their javascript at startup) - - -## Caution +## Cautions It's possible to create a loop with UE, and that currently isn't detected ([issue](https://github.com/chrisgoringe/cg-use-everywhere/issues/6), [PR for ComfyUI](https://github.com/comfyanonymous/ComfyUI/pull/1652)). If you get a RecursionError that's probably what you've done. Remember, *every* unconnected input gets connected to the UE output, even optional ones... you might want to use `Anything Everywhere?` nodes if this is a problem. +Bypassing and disabling nodes works, but with one catch. If you have a UE nodes that does matching (`Anything Everywhere?` and `Prompt Everywhere`) and you bypass the node it matches to, the link won't be made. So + +|If you use a ? node to send to a node...|...and bypass the recipient, it doesn't get connected | +|-|-| +|![1](docs/bypass_catch1.png)|![2](docs/bypass_catch2.png)| + +This is unlikely to be fixed, but should be fairly easy to avoid! # Deprecated nodes diff --git a/docs/bypass_catch1.png b/docs/bypass_catch1.png new file mode 100644 index 0000000..e0a1c5b Binary files /dev/null and b/docs/bypass_catch1.png differ diff --git a/docs/bypass_catch2.png b/docs/bypass_catch2.png new file mode 100644 index 0000000..526f500 Binary files /dev/null and b/docs/bypass_catch2.png differ