Repository navigation
Reduce complete dirty-certificate replay time by 22.57% (no new κ) - #6279
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
Scoped implementation-preservation proof and measured engineering improvement: a per-certificate prefix trie reduces the complete PR5235 dirty-circuit replay median by 22.57%, while preserving every certificate predicate and equation. This contribution was produced by Nick Petalas. No new dirty-circuit theorem, supplier admission, selected-frontier change or kappa is claimed.
Only
research/cached-dirty-paths/changes. The source compiler, field layer, independent checker, fixtures and complete caller retain zxv / PR5235 credit, including its explicit repeated-path performance handoff and full classical/echo/phase/tensor lineage. The source package remains unchanged. The copied checker retains its attribution; the exact-delta gate admits only the cache edit.Evidence
BENCHMARK.jsonbinds the measured executables and thirteen inputs.Reproduce from the repository root with CPython 3.12.15 and no third-party dependencies:
The scientific verifier prints
PASS_CACHED_DIRTY_PATHS. Replace the absolute interpreter path with any CPython 3.12 executable. Timing samples include startup, source authentication, imports, the complete unchanged scientific caller and output. An untimed profile identifies repeated path evaluation as the changed work. Local timing is not a portable speed guarantee.Limitations
The cache trades extra sparse-row storage for fewer transitions: O(S(U+R)) additional retained space for S states, U distinct nonempty rooted prefixes and R requested roots. Larger callers need a measured memory budget. No transitions are saved when every requested prefix is distinct. The implementation is opt-in; no shared code is changed.
No selected-supplier scientific replay, physical/native admission, rounded execution, compiler closure, stock/tape/precision or analytic-recovery proof was run. Lean, Docker, independent human and formal proof-kernel review are unclaimed. All five selected framework assumptions remain.
Open next questions
Can one larger source-bound independent-symbol adapter retain this benefit within a stated peak-memory budget? First measure its actual rooted-prefix count and sparse-row sizes, then time the complete producer/checker path. A compatible real adapter and measured memory budget remain blockers. This contribution stops at the first verified complete-caller improvement.
Related open work
The completion refresh inspected all sixteen open account PRs created since the advance CLAIM, including this PR. None of the fifteen other PRs studies this exact PR5235 checker or reports its rooted-prefix traversal quantities. No shared-instance numerical disagreement was found; no additional computations were run to check other contributions. Adjacent replay improvements use separate fixtures:
No numerical performance comparison with those PRs is asserted. PR5235 is merged and remains the credited predecessor. The refreshed exact-object search found no other open PR on this checker. These are bounded overlap checks, not a global novelty guarantee.
CI notes
Both hosted checks,
planandreplay (research/cached-dirty-paths), passed at exact head586f36c90586a02d2f866081469649e44cf94fbdin run 41068115208. The completed checks were read once; separate run metadata confirms the matching head, completed status and successful conclusion.Every registered command also passed locally, together with the direct warning-strict normal/optimized replay, source-history/byte/mode checks, export integrity and read-only Git-free replay. Hosted replay certifies the registered finite checks and source bindings. It does not certify the local timing percentage, independent human review or a formal proof-kernel theorem. No local package verification obligation is left unrun.
Advance claim: #1166 (comment)
Merge Danger
Door: two-way. Blast radius: isolated. Removing this opt-in package restores the prior behavior; frozen inputs and selected-frontier files are unchanged.