Repository navigation
Audit: independent replay confirms #681's 1,484-operation raw factory saving (no new kappa) - #802
Merged
Conversation
… factory saving (no new kappa)
Contributor
|
100-PR batch review at Merged to Reviewed scope: Independent PR681 raw-factory replay and 1,484-macro saving; unrelated bundled bit work not validated by this command. 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. |
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.
Independent audit of #681 by Alluri (@alluri-ritual), pinned at head
e536f6fc8aad65adef8ac4682d13625598a5c3f9. It covers #681's own contribution, the 87→34-gate replacement in all 14 raw PR622 coordinate-total windows. The bundled #696 lowering was audited in #783. Credit for the construction belongs to #681.Verdict: the 1,484-operation saving is confirmed exactly, for the raw constructor word. It does not reach the selected providers and does not change κ.
Exact base:
66849e1475c930fd7732b1a0067ffee7e1e95f60(currentmain, also #681's merge base). Onlyresearch/community-pr681-audit/is added.Reproduce
Expected:
It uses the standard library, runs offline, takes about 30 s and imports no #681 code. The two templates are transcribed from #681's
CIRCUIT.json(sha256f64a9e88…9f71).Findings
The replay runs the unchanged
mainconstructor directly and finds the 87-gate template by injective relabeling. It finds exactly 14 windows on disjoint registers, without usingBLOCKS.json. It splices the witness and rebuilds the cleanup by the literal inverse rule, after checking that rule against the old cleanup. Exact linear forms of all 716 registers match afterA,A+BandA+B+cleanup. The inherited native audit, emitter and local auditor frommainpass on both words.certificates/selected-result.jsonand the selected CK/JP8 provider words unchanged, and it does not transport the edit into selected recipes. The 1,484 are raw-constructor operations, not selected-factory operations. Scalar primitives enter only the finite invoice (Audit: independent recount confirms #696's 22–25% primitive saving (no new kappa) #783). κ stays51110230636731/31250000000000000.Limits
make verify,make frontier626-verify, Lean and #681's hosted replay were not run. #681's localverify.pyand the consolidation's localverify.pyboth pass here. The five bundled R420 bit-supplier packages were not audited. The prefix bounds come from the inherited auditor and were not re-derived.Rohan Guliani (@Rohansguliani), with substantial AI assistance. New files are under the repository's Apache-2.0 license.