comfyanonymous / ComfyUI

The most powerful and modular stable diffusion GUI, api and backend with a graph/nodes interface.
https://www.comfy.org/
GNU General Public License v3.0
43.11k stars 4.55k forks source link

Creating a loop (not submitting workflow) causes an uninterruptible error. #2077

Open GalaxyTimeMachine opened 8 months ago

GalaxyTimeMachine commented 8 months ago

I accidentally created a loop by taking the text output of a node and feeding it back into the input of the same node. This creates a pop-up error with an "OK" button, but clicking on the button just cycles the error back again. There is no way to rectify the error because the UI is unresponsive while the error show. image

SmokeyRGB commented 7 months ago

Seconding, really annoying if you've worked on a workflow for some time and are messing up all your work by making this mistake.

Only way to get rid of the alert box is to close the window, loosing all progress made on the workflow.

GalaxyTimeMachine commented 7 months ago

Even closing the window doesn't work because the same workflow reloads when you open it again. The only way I could clear it was to clear the browser cache, and so I lost all of my node templates as well :(

Smoky78 commented 7 months ago

Are you using UE Nodes - cg-use-everywhere ? If so, this caused this exact error with me. Uncommenting the loop checking section in "ComfyUI_windows_portable\ComfyUI\custom_nodes\cg-use-everywhere\js\use_everywhere.js", unlocks the ui and you can correct things.

GalaxyTimeMachine commented 7 months ago

Yes, I am using the "everywhere" nodes. Perhaps this bug needs to move over to there.

SmokeyRGB commented 7 months ago

Same, seems to be a problem with the UE-Nodes then. I was wondering why they would randomly introduce such a weird way of error-avoidance at this point. Thanks @Smoky78

chrisgoringe commented 7 months ago

Thanks for raising the bug across at UE - fixed in the latest update.

mr-bob-chang commented 3 months ago

I didn't use the cg-use-everywhere node, but I still run into this problem, This problem occurs when connecting ApplyControlnet for Inpaint to FaceDetailer node. 企业微信截图_20240408112150

chrisgoringe commented 3 months ago

@mr-bob-chang posting the workflow, or at least a screenshot, would be helpful.

note that you can turn the loop checking off in the main settings: Screenshot 2024-04-08 134725

YellowjacketXVI commented 1 month ago

I accidentally created a loop by taking the text output of a node and feeding it back into the input of the same node. This creates a pop-up error with an "OK" button, but clicking on the button just cycles the error back again. There is no way to rectify the error because the UI is unresponsive while the error show. image

You a node creating a loop as in something that should be another node is actually happening after and creating a loop, just double check all you nodes, make sure the latent is linear in one direction