Skip to content

refactor: abstract ComfyUI boundary into core/platform.py#128

Closed
AEmotionStudio wants to merge 3 commits into
mainfrom
refactor/platform-abstraction
Closed

refactor: abstract ComfyUI boundary into core/platform.py#128
AEmotionStudio wants to merge 3 commits into
mainfrom
refactor/platform-abstraction

Conversation

@AEmotionStudio

Copy link
Copy Markdown
Owner
  • Create core/platform.py with adapters:
    • get_models_dir(subdir): replaces folder_paths.models_dir
    • free_comfyui_vram(): replaces comfy.model_management calls
    • load_torch_file(path): wraps comfy.utils with torch fallback
  • Migrate 6 core modules to use platform adapters
  • core/ no longer imports folder_paths or comfy.* directly (except mmaudio's distinct unet_offload_device API)

- Create core/platform.py with adapters:
  - get_models_dir(subdir): replaces folder_paths.models_dir
  - free_comfyui_vram(): replaces comfy.model_management calls
  - load_torch_file(path): wraps comfy.utils with torch fallback
- Migrate 6 core modules to use platform adapters
- core/ no longer imports folder_paths or comfy.* directly
  (except mmaudio's distinct unet_offload_device API)
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown

⏱️ Watchdog timed out

Jules session did not complete within 30 minutes.

View session

@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown

⏱️ Watchdog timed out

Jules session did not complete within 30 minutes.

View session

@AEmotionStudio

Copy link
Copy Markdown
Owner Author

Squash-merged locally with safe_load=False fix for SAM3

@AEmotionStudio AEmotionStudio deleted the refactor/platform-abstraction branch March 2, 2026 21:45
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown

⏱️ Watchdog timed out

Jules session did not complete within 30 minutes.

View session

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