Repository navigation
Conversation
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 PR4345 literal header-census command spends most of its time constructing conjugated requests with dense products. A new source-bound kernel uses the requests' rank-one/rank-two factors instead. Local median command time falls 47.30%, from 9.341 s to 4.923 s, with identical complete output.
Claim type: scoped implementation-preservation proof with finite conformance and local performance evidence. The contribution was produced by Nick Petalas. Credit Ryanson Jonathan / pinterbanget for PR4345's complete engine and the retained header/BIT lineage. The conjugation identities are classical. Existing counts are conformance outputs, not new mathematical results or audit deliverables. No new kappa or selected-frontier change.
Only
research/low-rank-header-conjugation/is added. The authenticated predecessor source is unchanged. The scope update records this implementation target.Evidence
BENCHMARK.jsonbinds the exact source, adapter, kernel, flags and raw samples. Compilation is excluded; the full census process and final output are timed. Measurements describe this macOS arm64 / Apple clang 21 host, with other research processes active.PASS_LOW_RANK_HEADER_CONJUGATION,PASS_REGISTERED_RESEARCH_PACKAGEand metadata/pin checks pass. Committed-byte/source-history validation passes ate808800e784756a0eabf438e269bbe5005a0c75bfor ten package files plus the declared C input. C builds use-O2 -std=c11 -Wall -Wextra -Werror -pedantic.From the repository root, with CPython 3.12 and a C11
cccompiler:Another CPython 3.12 path can replace the displayed interpreter. Temporary C sources and binaries use an external temporary directory and are removed. Set
TMPDIRto choose it. Timing is optional and has no CI performance threshold.Limitations
This is a verifier implementation improvement, not independent validation of the predecessor's counting theorems. The proof covers the eight stated requests; executable conformance covers the eleven retained domains. The legacy engine's larger-size counter and CLI limitations are outside scope. No selected cold replay, native/compiler, Lean or Docker build ran. The five framework assumptions remain. Independent human and formal proof-kernel review are unclaimed.
Open next questions
Can a real consumer supply low-rank request factors directly and use this kernel without first constructing dense requests? Start with one complete caller while retaining its field representation, independent inputs and every guard. That caller and its benchmark are missing. Optimization stopped after this first measured improvement.
Related open work
No open PR studying PR4345's engine or this low-rank header-conjugation kernel was found in the refreshed source/title/body searches.
The completion refresh inspected all ten account PRs still open and created since the CLAIM at 2026-11-02T21:22:08Z, including this PR. The other nine are #6406 through #6414. None computes a conflicting value for this same kernel or benchmark. No extra computation audited another contribution. Main README searches, recent account work and local reservations were checked; unpublished work and arbitrary historical diffs remain coverage limits.
CI notes
Both hosted checks,
planandreplay (research/low-rank-header-conjugation), passed at exact heade808800e784756a0eabf438e269bbe5005a0c75bin run 41529685209. The completed checks were read once: plan took 4m25s and replay took 4m30s. Run metadata independently confirms the matching head and successful conclusion. All local checks above passed. Hosted CI certifies the registered finite conformance and source bindings; it does not certify the local timing percentage, independent scientific acceptance or a proof-kernel check.Merge Danger
Door: two-way. One isolated package can be reverted.
Blast Radius: package. Shared tooling, predecessor sources and selected-frontier files are unchanged.