Conversation
flamboh
added this pull request to stack #113
September 25, 2026 10:02
Move the conformance oracle from chris-misa/maad 3ae7536 to b7bbb7d on main, which includes the merged atomic-full-sortout branch and the critical-region spectrum filter, keeping the local deltaQ = 1/8 patch. Goldens are regenerated from the patched b7bbb7d oracle. IPv4 structure is unchanged. Dimension rows are now ordered q = 0, 1, 2 and carry sd. The spectrum keeps only rows whose q lies in the critical region, from min(0, min q <= 0 with f > 0) to max(1, max q >= 1 with f > 0), and the alpha run is non-increasing (upstream a1 >= a2) rather than strictly decreasing. Rust treats alphas within 1e-12 as ties. Stored spectrum and dimension JSON change, so the MAAD contract version and the Rust JSON schema version are bumped.
flamboh
force-pushed
the
maad/03-pin-bump
branch
from
September 27, 2026 05:17
3e1e0c6 to
b590ac3
Compare
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.
Note
🤖 Claude Opus 5.5 on behalf of Oliver
ELI5
Updates our reference copy of the MAAD research code to the author's latest version and matches our Rust port to it.
Why
Upstream chris-misa/maad merged
atomic-full-sortoutintomainand then fixed spectrum estimation (b7bbb7d). Our conformance oracle was pinned to an older commit.What changes
main@ b7bbb7d, still with the one-linedeltaQ = 1/8patch. The oracle is the separately cloned upstream checkout described indocs/code/maad-conformance.md(/tmp/maad); thevendor/maadgitlink is unchanged by this PR.sd, as upstream. D1'ssdis0, meaning "not estimated".mixedfixture goes from 25 to 16 spectrum rows.Review path
cargo test -p atlantis-netflow-db --test maad_conformance— all five fixtures pass against the regenerated goldens.Optional oracle check: build the patched oracle per the guide, build Rust with
./scripts/build_maad_fast.sh, then run on a public fixture:Expected:
mixed: PASS total=128 prefixes=17 rows=structure:33,spectrum:16,dimensions:3and exit code 0.Edge cases and decisions
--auto-stop, IPv6 input, and the new Hotelling T² /--compare-structuretests are not ported; none are part of the compared IPv4 output.a1 >= a2stops early on last-bit noise in τ while Rust keeps all tied rows. Goldens and real windows agree; the validator still requires exact row counts, so use inputs with real spectrum curvature.Verification
bun run format,bun run lint,bun run typecheck,bun run test:db(includesmaad_conformance).mixedfixture against a freshly built patched oracle (b7bbb7d +deltaQ = 1/8): output matched the expected line above, exit 0.Made by Claude Opus 5.5 (with Opus 5.5 subagents) via Claude Code.