Skip to content

Add minimax_h3 to IMG_ARCH_LIST - #476

Open
GabrielSalerno03 wants to merge 1 commit into
city96:mainfrom
GabrielSalerno03:fix/minimax-h3-arch
Open

Add minimax_h3 to IMG_ARCH_LIST#476
GabrielSalerno03 wants to merge 1 commit into
city96:mainfrom
GabrielSalerno03:fix/minimax-h3-arch

Conversation

@GabrielSalerno03

Copy link
Copy Markdown

MiniMax H3 UNet GGUFs that set general.architecture = "minimax_h3" are rejected by the loader:

ValueError: Unexpected architecture type in GGUF file: 'minimax_h3'

ComfyUI has native H3 support (comfy/ldm/minimax/, comfy_extras/nodes_minimax_h3.py), and the Qwen3-VL text encoder already loads because qwen3vl is in TXT_ARCH_LIST — only the diffusion model side is blocked.

Of the two community repacks I tested, minimax_h3_fl2va_pruned_fp8_Q4_0.gguf declares minimax_h3 and fails, while MiniMax-H3-FL2VA-Q3_K_M.gguf declares wan and loads. So at present the list only accepts H3 files that are mislabeled.

Tested on ComfyUI 0.33.0 with the pruned fp8 Q4_0 repack; text-to-video and image-to-video both generate correctly with this change.

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