Align MegaMoE training with upgraded K-grouped TMA and layouts - #17
morgendave wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Risk: medium. Left a non-blocking comment; this automation did not approve and did not assign reviewers. Human review is needed: MegaMoE wgrad TMA/layout changes are above the low-risk auto-approve threshold and need a correctness check of the 3D store descriptor and training oracle alignment.
Sent by Cursor Approval Agent: Pull Request Approver
There was a problem hiding this comment.
Risk: medium. Left a non-blocking comment; this automation did not approve and did not assign reviewers. Human review is needed: MegaMoE wgrad TMA/layout isolation from global grouped alignment is above the low-risk auto-approve threshold and needs a correctness check of the store descriptors and training oracle alignment.
Sent by Cursor Approval Agent: Pull Request Approver


2026-09-21: wide-K wgrad recovery
Current head:
0be759300fd5d198ef211e452ccb857217c41a1a.48e4df7includes this exact ordinary update. Its EP4/262K ordinary BF16 and MXFP4 complete fixed-input backward tensors are byte-identical across K=16/64/16/64 on every rank, cosine=1/L2=0. All-expert independent FP32 wgrad oracle minimum cosine: 0.9999998386.Same physical 4×B300, EP4, 65,536 tokens/rank, H4096/I2048/E256/top-k6; native-API timing at the companion side-stack head with side disabled:
Direct F+B medians over 3×10 max-rank CUDA-event samples after warmup. W13/W2 wgrad+combine recovered from 39.79 to 13.48 ms BF16 and 39.28 to 12.50 ms MXFP4. Ordinary wgrad output dtype remains BF16. No transforms, AC, optimizer timing, full-model E2E, or ARM certification. Companion ordinary integration #50925 pins this head; #9 includes it and #44111 pins that stack.
Prior alignment validation and historical measurements (superseded heads)
Summary
Align the existing MegaMoE training bridge with the upgraded DeepJIT / K-grouped GEMM implementation on
02452d4(after #15/#16). This is an upstream-upgrade compatibility update, independent of the side-LoRA extension in #9.[N, M, group], matching the new K-grouped 3D TMA store. The old 2D descriptor traps atSM90_TMA_STORE_3D::copyon the upgraded kernel.Validation
02452d4; compute-sanitizer identifies the 3D TMA store.No scheduler, ring-sizing, model-lifetime caching, or side-LoRA behavior changes. No performance or ARM validation claim in this PR. The integration update must consume the native layout query as well.