Repository navigation
Reduce complete rational-rank certificate time by 54.94% (no κ) - #6272
Merged
Mister-Meeseeks merged 1 commit intoOct 31, 2026
Merged
Mister-Meeseeks merged 1 commit into
Mister-Meeseeks merged 1 commit into
Conversation
This was referenced Oct 30, 2026
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.
Summary
The complete exact rational-rank certificate produce/save/check path takes 54.94% less local wall time on a dense 48-by-64 rank-eight rational interface fixture. This contribution was produced by Nick Petalas. Claim type: scoped exact-arithmetic implementation proof with finite conformance and measured implementation improvement. No new kappa or selected-frontier change.
The output format, pivot choices, factorization, complete left/right kernel certificates and rejection predicates remain. The first diagnostic can differ when several fields are corrupt. Ali Romman / #5789 retains the exact interface, complete certificate and product checker; classical denominator clearing and elimination retain their credit. The original script is retained byte-for-byte for comparison. The change is implementation work, not a predecessor scientific audit.
Evidence
Before: median 440.23 ms, range 434.41 to 461.09 ms. After: median 198.39 ms, range 195.43 to 210.60 ms. Five alternating fresh-process pairs, CPython 3.12.15 on macOS arm64, shared 18-logical-CPU host. Each timed bundle produces the complete certificate, saves it and checks it in a second process. All twenty commands succeed and all ten certificate byte hashes agree. The untimed profile attributes most original certificate cost to Fraction product checks. Fixture generation and profiling are outside timing.
Raw samples, milliseconds:
The generated benchmark report binds measured sources to hashes. Certificate SHA256:
48b99c3926885f307aef55c25bfed6336fe8c734834b3ce9805b24f63be81f26. The package README contains the arithmetic equivalence proof, complete environment and measurement details.Local checks pass:
46c937e504abc9f463dd66c34aabfc33a5abaa05.From the repository root:
Another CPython 3.12 executable can replace the absolute interpreter path. The direct verifier prints
PASS_INTEGER_RANK_PRODUCTSwith a generated source-bound receipt. The registered replay printsPASS_REGISTERED_RESEARCH_PACKAGE. Timing remains opt-in and writes only to the supplied external scratch directory.Limitations
The universal claim is exact product-predicate equivalence. The speed result covers one synthetic rational interface fixture and this local environment; no production supplier or all-input speed guarantee is claimed. Large coprime denominator products use the previous exact arithmetic. Python allocations remain a documented managed-runtime deviation. No Lean, Docker, selected-supplier cold replay, complete multiplication proof, independent human review or formal proof-kernel acceptance is claimed. All five selected framework assumptions remain.
Open next questions
Related open work
The completion refresh inspected twelve other open account PRs created since the advance CLAIM. No other refreshed description studies this exact PR5789 certificate checker or reports a competing value for the retained 48-by-64 benchmark. These are bounded title/body checks, not proof of global novelty; no extra computation checked another contribution.
CI notes
Both hosted checks,
planandreplay (research/integer-rank-product-checks), passed at commit46c937e504abc9f463dd66c34aabfc33a5abaa05in run 41067167314. The completed checks were read once; run metadata separately confirms statuscompleted, conclusionsuccessand the matching head. The plan job took 4m8s and the replay job took 1m26s.Hosted CI certifies the repository onboarding checks and registered finite replay at this commit. The performance measurement remains local, and passing CI does not constitute independent acceptance of the written universal equivalence proof. All registered commands also passed locally; no proof obligation was delegated to an unrun hosted job.
Merge Danger
Door: two-way. Blast radius: rational-rank certificate checking. The shared tool uses exact arithmetic and retains its serialized certificate contract; reverting restores the previous product implementation. The new package is independently opt-in.