Skip to content

feat(parity): per-pair flip-position dump for the margin bootstrap - #120

Merged
ronaldtse merged 1 commit into
mainfrom
feat/margin-positions
Sep 1, 2026
Merged

feat(parity): per-pair flip-position dump for the margin bootstrap#120
ronaldtse merged 1 commit into
mainfrom
feat/margin-positions

Conversation

@ronaldtse

Copy link
Copy Markdown
Contributor

run_margin_analysis gains dump_positions: one JSONL record per pair (tokens, flip positions, reference margins at those positions). MarginReport is unchanged (schema-additive diagnostic). The test reconciles the dump's aggregates with the report. This is the enabling half of TODO.training-work/05 — shipped-vs-head32 flip-rate CIs; the Modal margin re-runs follow.

…ap needs

run_margin_analysis(dump_positions=...) writes one JSONL record per
pair (tokens, flip positions, their reference margins) beside the
aggregate margins JSON. MarginReport stays schema-identical; the dump
is diagnostic. Enables the TODO.training-work/05 shipped-vs-head32
flip CIs; aggregate flip counts in the dump must reconcile with the
report (tested).
@ronaldtse
ronaldtse merged commit cb8902d into main Sep 1, 2026
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