Repository navigation
Tooling: 3.18× faster production R420 xcheck.rref via integer elimination with identical receipts - #2263
Open
VictorLiwentao wants to merge 2 commits into
Open
Tooling: 3.18× faster production R420 xcheck.rref via integer elimination with identical receipts#2263VictorLiwentao wants to merge 2 commits into
VictorLiwentao wants to merge 2 commits into
Conversation
…tion with identical receipts Wire the proved primitive-integer RREF from research/r420-rref-performance into production research/e8-bit-r420/code/xcheck.py. Identical check and mutants stdout, 672 frame RREF/nullspace comparisons, MANIFEST digest update. No new kappa. Prepared by Wentao Li with substantial AI assistance.
…provenance After production integer rref, keep the Fraction-era digest as provenance so research-packages CI does not fail working source drift when xcheck.py changes. Live VERIFY inputs drop that path; triple-bases check_records still PASSes. Prepared by Wentao Li with substantial AI assistance.
This was referenced Oct 15, 2026
Open
Contributor
|
Maintainer review of
This is a production-integration hold, not a rejection of the speedup or its author benchmark. Baseline output equality and original attribution are valuable; they need to survive together with existing reproduction commands. Our supplemental evidence is retained in the efficient1300 audit as |
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.
Change and scope
At base
0fdd7d82de03e43c9e5a1000dfbcd380d1b687d2, productionresearch/e8-bit-r420/code/xcheck.py:rrefstill used Fraction row reduction on every nonempty frame. The opt-in adapterresearch/r420-rref-performance/already proved Aman’s PR367 primitive-integer elimination with identical check/mutants receipts. This PR wires that equivalent RREF into the production checker and updatesresearch/e8-bit-r420/MANIFEST.sha256to the new digest.No certificate κ pins change. The historical Fraction-era
xcheck.pydigest inresearch/r420-triple-bases-seed01/SOURCES.jsonis demoted to provenance (and dropped from liveVERIFY.jsoninputs) so research-packages CI does not fail working-source drift after the production rewrite. Other historical inventories that record the Fraction-era digest without a registered VERIFY matrix are left as provenance.Tooling improvement; no new kappa. Selected frontier untouched.
Result
On Linux / CPython 3.12.3, five alternating fresh-process pairs gave stage-median-sum speedup ≈3.18× (check ≈2.94×, controls ≈3.21×). Timing is host-descriptive and is not a correctness gate. Check and mutants stdout digests match the baseline checker; all 672 frame RREF/nullspace comparisons agree.
Package:
research/r420-rref-production/.Tip:
99cb4fbf2b55a5e8265e066a1e4793d92f9b5077.Validation
Expected:
Default verify is read-only. Temp-copy sign-flip foil rejects via subprocess failure. Package gates under
-Ostill launch assertion-enabled children. Fullmake verify, selected cold replay, Lean, and multiplication-theorem obligations were not rerun for this isolated checker change.Overlap and credit
Closest priors (different delivery):
research/r420-rref-performance/(opt-in integer RREF; this is the production wiring), Aman PR367 /research/frame-algebra-performance, CrocSwap/integer-mult-bounds#756, CrocSwap/integer-mult-bounds#2257 (same production-promotion + provenance-demotion pattern). No competing open PR on productionxcheck.rrefat refresh.Credit eumemic (PR543 checker/harness/word), Aman (PR367 integer rows), and the authors of
research/r420-rref-performance/. Prepared by Wentao Li (VictorLiwentao, @Is__Victor) with substantial AI assistance, under Apache-2.0.