Repository navigation
[Model] Add DeepGEMM MegaMoE support to GLM with compressed-tensors MXFP4 - #55275
tlrmchlsmth wants to merge 38 commits into
Conversation
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
|
This pull request has merge conflicts that must be resolved before it can be |
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
|
✅ Triggered Buildkite CI #88564 for commit |
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
|
/ci run |
|
✅ Triggered Buildkite CI #88670 for commit |
|
This pull request has merge conflicts that must be resolved before it can be |
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com> Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
|
/ci run |
|
✅ Triggered Buildkite CI #89108 for commit |
|
This pull request has merge conflicts that must be resolved before it can be |
Purpose
Extend the
deep_gemm_mega_moebackend to DeepSeek-derived GLM modelimplementations without changing compressed-tensors NVFP4 weights into a
different FP4 encoding.
This change:
MTP paths;
lossy NVFP4-to-E2M1/UE8M0 conversion;
semantics;
model files and the existing DSv4 tests are unchanged;
rejection with focused tests.
Duplicate-work check
This does not duplicate the open related work:
not provide these GLM/DeepSeek model adapters.
external DeepGEMM package; this PR uses the existing FP8-activation/FP4-weight
MegaMoE path.
MegaMoE on SM100.
Test plan and results
Local checks after splitting the shared implementation from DSv4:
The CPU tests cover checkpoint-format selection, NVFP4 rejection, mocked
BF16/MXFP4 loading and finalization, expert mapping, and deferred-reduction
scaling. Non-compressed-tensors quantization configurations are not rejected
by the MXFP4 format predicate.
DSv4 migration is kept separately on
feature/dsv4-shared-megamoe,
stacked on this PR. No follow-up PR has been opened. Kimi K3 file changes are
omitted from both branches.
The following results predate this extraction; no new GPU or model-accuracy
evaluation was run for the split.
GB200 kernel validation performed earlier on the PR:
similarity greater than 0.9996.
intermediate size 2048, 256 experts, top-k 8): all ranks produced finite
outputs.
MXFP4 performance validation used
RedHatAI/GLM-5.2-MXFP4xMXFP8on two GB200nodes with TP8/EP8. At concurrency 64, DeepGEMM MegaMoE reached 927.8 output
tokens/s, compared with 562.2 for TP8, 540.2 for EP8 with AGRS and CUTLASS
MXFP4, and 237.8 for EP8 with DeepEP-v2 and CUTLASS MXFP4. All 2,269 recorded
requests across the four runs completed successfully. These are single-run
client-side measurements, not repeated trials; Prometheus/DCGM series were not
exported.
No NVFP4 model evaluation is claimed: compressed-tensors NVFP4 is deliberately
unsupported by this PR because conversion into DeepGEMM's different FP4
representation showed unacceptable source-level error.
AI assistance
AI assistance was used to implement, debug, test, and draft this change.
Human review of the latest split and relevant GPU/model validation are pending.