Skip to content

Update MiniMax H3 for current ComfyUI - #7

Merged
buqi-code merged 6 commits into
mainfrom
fix/comfyui-0.37
Sep 28, 2026
Merged

buqi-code merged 6 commits into
mainfrom
fix/comfyui-0.37

Conversation

@buqi-code

Copy link
Copy Markdown
Owner

Summary

  • restore normal ComfyUI custom-node discovery and add current official workflow examples
  • port Ulysses SP to the current H3 layout, guide/mask, attention-container, PDD and audio-carry contracts
  • synchronize static LoRA patches and make NCCL startup/recovery fail fast
  • retire the unsafe parallel VAE handover while preserving the legacy node as an official-decode alias

This integrates and preserves the authorship of the compatibility, LoRA and rendezvous work contributed in #6, then updates it for the current ComfyUI H3 API.

Closes #1
Closes #2
Closes #3
Closes #5

#4 is intentionally left open until the reporter confirms the startup-probe fix on the original dual-5090 topology.

Validation

  • 46 CPU tests passed; one GPU-only discovery test skipped as designed
  • two-rank CPU/Gloo T2V, I2V, audio-guide, reference, masks and PDD parity passed
  • real 2×L20 INT8 DiT SP2 parity passed: video relative error 1.199e-6, audio bit-exact
  • real loader/Gloo/NCCL creation, all-reduce/all-to-all startup probe and clean shutdown passed
  • T2V, first-frame I2V, first+last-frame I2V, AddGuide and R2V completed through current ComfyUI with SP2
  • Python compile, workflow JSON parsing and diff checks passed

Not validated here

  • four-GPU execution (the current host exposes two GPUs)
  • a real Turbo LoRA checkpoint (protocol/unit coverage is included; GPU workflow remains opt-in)

RagnarokChan and others added 6 commits September 29, 2026 02:38
Restore installability and align sequence parallelism with current H3 conditioning, attention, PDD, and failure-recovery contracts while adding executable workflows and release gates.
Force ComfyUI into CPU mode before model-management imports so CUDA-enabled wheels remain testable on driverless GitHub runners.
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.

Compitability with latest ComfyUI 这个功能是没实现吗 Doesn't seem to load? Missing __init__.py does buqi support int8 h3?

2 participants