Repository navigation
Tooling: replay balanced Fano 2.91× faster with identical exhaustive streams - #756
Merged
Mister-Meeseeks merged 1 commit intoOct 14, 2026
Conversation
VictorLiwentao
force-pushed
the
tooling/seed91-balanced-fano-columns
branch
from
October 13, 2026 05:33
d77ccc4 to
4fad623
Compare
Contributor
|
100-PR batch review at Merged to Reviewed scope: Packed balanced-Fano enumeration: order-preserving column encoding and first-match reconstruction; complete half-stream differential tests plus dense independent controls. 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 was referenced Oct 14, 2026
Merged
cursor Bot
pushed a commit
to VictorLiwentao/integer-mult-bounds
that referenced
this pull request
Oct 14, 2026
Identity stands; absolute ~23ms residual after CrocSwap#2001 is below the CrocSwap#756 tooling bar (noisy independent rebenches near ~1.5×).
This was referenced Oct 14, 2026
Merged
Merged
This was referenced Oct 15, 2026
Open
Open
Open
Merged
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 historical balanced-Fano checker transposes a complete binary matrix at every meet-in-the-middle leaf. This tooling improvement propagates packed columns with exact Python integer operations and reconstructs a retained prefix matrix only when a witness is found. Both exhaustive streams, first-match behavior, witnesses, budgets and rejection checks are preserved. No new κ or search-domain claim.
Five sequential, alternating fresh-process pairs on CPython 3.13.2 / macOS arm64 measured median search time 5.969 → 2.051 seconds (2.91×) and peak process RSS 162,381,824 → 113,180,672 bytes (30.3% lower). Every sample matches every original result field. These measurements cover this fixed historical check, not the selected supplier replay or multiplication runtime.
Validation:
6^14 = 78,364,164,096local assignments via meet-in-the-middle.PYTHONDONTWRITEBYTECODE=1 make verify-tests: 555 tests in 85 isolated modules and 20 patch checks pass. The six new tests also pass under-O; the affected audit passes under-S; selected-record and whitespace checks pass.upstream/are unchanged.Reproduce with standard-library Python 3.11+ (benchmark RSS reporting supports macOS/Linux):
Expected:
PASS_IDENTICAL_COMPLETE_RESULTS.research/balanced-fano-performance/README.mdcontains the equivalence argument and review map;RESULT.jsonretains all samples and exact source pins;VALIDATION.jsonrecords coverage. Fullmake verifyand the selected supplier cold replay were not rerun for this isolated historical-checker change. Remote CI is separate from local validation.Exact base:
66849e1475c930fd7732b1a0067ffee7e1e95f60. OS-random assignment seed 91, verification engineering. No pinned sprint issue existed, so the claim is local. Final overlap review covered 567 open PRs through #748; nearby algebra, geometry, footprint and streaming optimizations concern other checks and supply no code here.Credit Douglas R. Colkitt's original balanced-Fano checker, scoped negative result and dense controls. Prepared by Wentao Li with substantial OpenAI Codex assistance, under Apache-2.0; Contributor profiles: GitHub: VictorLiwentao, X: @Is__Victor. Detailed predecessor attribution is in
NOTICE.md. This changes no inherited mathematical assumptions and is not a formal proof of the full multiplication theorem.