Repository navigation
[WS1][MiniMax-H3] h3_arch_fingerprint, h3_operator_trace, h3_trajectory_artifact: pinned architecture, strict-run ledger and sealed trajectory (RFC #420 stack 1/5) - #567
Open
fusheng-ji wants to merge 8 commits into
Conversation
fusheng-ji
requested review from
Flink-ddd,
KJLdefeated,
bitborne,
inaniloquentee and
ryankert01
as code owners
October 22, 2026 12:18
|
Important Review skippedToo many files! This PR contains 140 files, which is 40 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry.
|
This was referenced Oct 22, 2026
Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com> (cherry picked from commit eaba5efe3df971ea9d1a969f53e48f09e09ed176) Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com> (cherry picked from commit 7da0574)
Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com> (cherry picked from commit b45190d08fc033c7e63ad821d3918eb32dd454f6) Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com> (cherry picked from commit b5536d6)
Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com> (cherry picked from commit ad1925045b522772bf56f8468b23cca47f04156b) Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com> (cherry picked from commit f9041e8)
Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com> (cherry picked from commit b44540e)
…block 0 node-by-node and 50-block/generation gates Squashes the one-block branch (its row-by-row history: timestep sinusoid, RMSNorm, AdaLN rows, sequence/tensor parallel norm rows, final AdaLN, one block, 50-block qualification) onto the current test-h3 so the pull request carries a reviewable, signed history. RFC RL-Align#420. Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com>
…ody) Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com>
fusheng-ji
force-pushed
the
feat/h3-fingerprint-trace-trajectory
branch
from
October 22, 2026 21:04
b44540e to
eab901b
Compare
fusheng-ji
added a commit
to fusheng-ji/RL-Kernel
that referenced
this pull request
Oct 23, 2026
…#567 RL-Align#568 RL-Align#570 RL-Align#571 RL-Align#574 RL-Align#575 Test summaries (tests.json), runner outputs and figures measured on NVIDIA B200 (sm100), driver 580, CUDA 13.0, torch 2.13.0+cu130, MiniMax-H3@42ed227, clean trees at the PR heads eab901b, 3cf51bf, 6de320d, eb7840b, af3dcae and 4f7c5d1. Figures use B200 axis labels. No host names, paths or scheduler identifiers. Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com>
fusheng-ji
added a commit
to fusheng-ji/RL-Kernel
that referenced
this pull request
Oct 23, 2026
…jectory diag summary - h3-fingerprint-trace-trajectory-h100/tests.json (77 passed, eab901b) - environment.json for RL-Align#574/RL-Align#575 H100 adapter reproducibility pins - dual_trajectory_diag_summary.json (H100/B200 diag strict 0/16 on seeds 11/22/33) Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com>
This was referenced Oct 25, 2026
Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com>
…ctory Signed-off-by: Wenbo Ji <36562829+fusheng-ji@users.noreply.github.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
rl_engine/models/minimax_h3/fingerprint.py,arch_fingerprint.json,pinned_checkpoint.json): the 33B H3 as a table of50 transformer blocks x 17 nodes, 2 token-refiner blocks x 14 nodes, the final norm and 638 tensors (names, shapes, dtypes), pinned to
MiniMax-H3@42ed227.tools/validation/models/h3_fingerprint.pychecks a checkpoint snapshot against it;tools/weights/snapshot_h3_checkpoint.pywrites the snapshot.operator_trace.py): every node of a run records which implementation bound it (row,interim,reference,native) and a strict runis only accepted when the ledger is complete and every node is
row.trajectory.py): a sealed record of a denoising trajectory (per-step velocity / mean, per-modality hashes, optional tensors) with adrift_reportthat names the first step, kind and modality that differ.Prior art & reuse decision
No existing library offers a pinned per-node architecture table or a ledger of which kernel computed each node: diffusers, vLLM-Omni and SGLang implement the model but
do not expose this; verl, Miles and Vime have no H3 diffusion operators. Nothing to reuse; these are small pure-Python pieces with a CLI each.
Results (H100 / Hopper)
No operator numerics here, so no accuracy/performance charts apply. Measured on NVIDIA H100 (93 GiB, sm90), driver 580.173.02, CUDA 13.0, torch 2.13.0+cu130, pinned
MiniMax-H3@42ed227weights, at headeab901bd.H100: 77 passed (77 collected) —
tests.json.Results (B200 / Blackwell)
B200: 77 passed (77 collected) —
tests.json.Tests
python -m pytest tests/models/minimax_h3/test_h3_arch_fingerprint.py tests/models/minimax_h3/test_h3_operator_trace.py \ tests/models/minimax_h3/test_h3_trajectory_artifact.py -q # 77 passedblack / isort / flake8 clean (pre-commit settings).
Scope / Known limitations
Not part of this PR: video projections (#562), O / gate_up / down GEMMs (#554, #535), full attention (#553), ROCm, SGLang adapter, pipeline parallel. The ledger accepts only
rowbindings; the interim stand-ins other contributors still own are reported, not hidden.