Skip to content

Commit

Permalink
fix: ComfyUI stuck for 10mins issue (#647)
Browse files Browse the repository at this point in the history
  • Loading branch information
thundergolfer authored Mar 12, 2024
1 parent c4c6d4e commit 8762563
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion 06_gpu_and_ml/comfyui/comfy_ui.py
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,11 @@ def spawn_comfyui_in_background():
# Restrict to 1 container because we want to our ComfyUI session state
# to be on a single container.
concurrency_limit=1,
timeout=10 * 60,
# Extend idle timeout of container so that you can switch tabs, read a
# quick article, and come back to resume work on same container.
container_idle_timeout=10 * 5,
# Timeout requests after 60s
timeout=60,

This comment has been minimized.

Copy link
@ekzhang

ekzhang Mar 12, 2024

Member

timeout doesn't work properly with allow_concurrent_inputs` right now, it will actually delete the entire runner which is non-ideal

)
@modal.asgi_app()
def web():
Expand Down

0 comments on commit 8762563

Please sign in to comment.