Repository navigation
Tooling: skip exact zero row updates in cyclic-inverse replay - #736
Merged
Mister-Meeseeks merged 2 commits intoOct 14, 2026
Merged
Mister-Meeseeks merged 2 commits into
Mister-Meeseeks merged 2 commits into
Conversation
Contributor
|
100-PR batch review at Merged to Reviewed scope: Opt-in zero-elimination adapter; exact finite predecessor receipt retained. Runtime evidence is local, not a new theorem or production default. The pinned scoped command passed; the batch ledger contains its command, input hashes, environment and log. This incorporates the research/tooling contribution and does not change the selected κ or discharge the inherited multiplication-theorem assumptions. Batch review, attribution and evidence. Thank you for the contribution. |
cursor Bot
pushed a commit
to VictorLiwentao/integer-mult-bounds
that referenced
this pull request
Oct 15, 2026
Promote thakrarsagar CrocSwap#736 / research/cyclic-zero-elimination into research/prime-power-cyclic-inverse/verify.py with identical EXPECTED.json receipts. Vendor pairwise baselines for the historical opt-in adapter. Prepared by Wentao Li with substantial AI assistance. Co-authored-by: Victor Li <VictorLiwentao@users.noreply.github.com>
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.
The incorporated PR621 verifier performs exact cyclotomic arithmetic even when an elimination coefficient is zero. This source-pinned adapter inserts a two-line guard in memory and runs every original check. The eight-case receipt remains byte-identical. Historical source bytes and default verification stay unchanged.
Status: tooling improvement; no new kappa. Seven alternating full-process pairs on each of two local interpreters measured 24.6–26.2% lower child CPU medians. Wall medians also improved, but one pair was slower; all samples and the preliminary probe are disclosed. This is a scoped result for the existing eight cases, not a full-frontier speedup.
Reproduce with Python 3.11+ and the standard library on macOS or Linux:
Use a fresh output path. Expected:
PASS_CYCLIC_ZERO_ELIMINATION. The JSON retains every timing, source pin, platform, and control result.replay.pyruns only the faster complete replay and accepts the predecessor's--outputoption.Validation:
EXPECTED.jsonbyte for byte, including the original unit-root and missing-conjugation controls for every case./tmpalso match on Python 3.11 and 3.14. A separate Codex agent reviewed the guard and ran focused controls under optimized Python.make selected-record-checkandgit diff --checkpass. The full repository suite, cold frontier replay, and optional SymPy engine were not rerun for this optional adapter.Base:
66849e1475c930fd7732b1a0067ffee7e1e95f60. Predecessor: Ali Romman's incorporated #621 at520774462b19b29af41554b74def31f9d01e997e; exact source and receipt hashes are inSOURCE.json. Scott Woolsey Biggart's pending #668 independently extends resultant coverage through radix 27 using CRT. This work keeps the predecessor's domain and rational-field method and has no dependency on that PR. The PR queue was refreshed through #732; #727's owner-footprint optimization is a separate checker.All additions are confined to
research/cyclic-zero-elimination/. Seed 94, local claim, explanation of the guard, raw evidence, and inherited limits are included. No pinned sprint issue was available. Contributor: thakrarsagar, with substantial OpenAI Codex assistance and a separate AI code review. Apache-2.0; predecessor attribution is retained. This does not prove the full analytic or multiplication theorem, or resolve paid preparation, rounding/restoration, or fixed-tape realization.