Skip to content

fix(memory): prevent VramOrchestrator eviction latch and guard ComfyUI offline free calls - #106

Merged
spelech merged 1 commit into
mainfrom
fix/vram-orchestrator-eviction-latch
Nov 1, 2026
Merged

spelech merged 1 commit into
mainfrom
fix/vram-orchestrator-eviction-latch

Conversation

@spelech

@spelech spelech commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Check IsComfyUiHealthyAsync before posting to :8188/free in VramOrchestrator.FreeComfyUiVramAsync to avoid repeated SocketException 10061 errors when ComfyUI is offline.
  • Prevent one-way eviction/hibernation latch in VramOrchestrator.EvaluateCacheLifetimesAsync by resetting _tier2Hibernated when an engine process is active again and allowing RAM pressure eviction even after Tier 1 idle eviction.
  • Clean up and dispose exited Process handles in AiEngineManager.GetActiveProcesses() and engine process properties, and record activity via lazy VramOrchestrator resolution.
  • Record engine activity in WorkflowEndpoints (/api/3d/generate, /api/video/generate, /api/audio/generate) and LocalLlmMcpTools, and wrap HttpResponseMessage and StringContent instances in using statements.
  • Add unit tests in VramOrchestratorTests.cs covering ComfyUI offline guard and eviction/hibernation latch recovery.

@spelech
spelech merged commit 91c96dc into main Nov 1, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant