Repository navigation
Remove three endpoint reversals from physical convolution (no new κ) - #6024
Merged
Mister-Meeseeks merged 1 commit intoOct 28, 2026
Merged
Conversation
This was referenced Oct 27, 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 program-refinement proof with finite physical corroboration. The complete normalized convolution of Jose Manuel / jmbarrancoml's #3709 can omit the three endpoint reversal routines inherited from #3640, retaining bit-for-bit output equality and the same borrowed-port contract.
Thus the common frequency permutation cancels through the actual record-local polynomial consumer. For axis bit lengths a_i and P=p+sum a_i, this removes exactly
3 ceil(E/b)outer sparse-router invocations, whereE=sum floor(a_i/2)andb=2^floor(997 floor(log2 P)/1000). The nonidentity-B four-address fixture removes three. There is no measured runtime factor, changed recursive supplier histogram, new kappa or selected-frontier change.This answers #3640 Open next question 2 using #3709's later physical product. Compatible DIF/DIT ordering is classical and already appears in the upstream raw interface, androolloyd's indexed-read #673 integration and #3709's inner coefficient FFT. The new contribution is its executable refinement of this particular outer fixed-tape caller, the complete rounded-output/ownership argument and exact batching reduction. Predecessor theorems are cited dependencies, not re-audited deliverables. Produced by Nick Petalas.
Evidence
ceil(E/b)nonempty routing batches each. After: the new private Fourier adapter omits those three routines inside the unchanged complete Convolution caller.ad81a1219dc838dd3cd05b2aa4fb93aa80b7dee0edc77c4e0fb3b149482da8e9. It retains six roots, three diagonals, three lifts, three truncations, the polynomial product and final M scale, with zero outer reversals. Borrowed data/flags/counts, scratch erasure and protected width-port rejection pass.d1f912d9c431414c7ea639baab0415c7183958d7. The official export checks all 13 declared inputs. Read-only, empty-PATH, Git-free optimized algebra replay passes; deliberate source corruption rejects, and all 24 exported files retain their original bytes after the control.From the repository root:
Any CPython 3.12 executable can replace the local path. Expected markers are
PASS_REVERSAL_FREE_CONVOLUTION,PASS_REGISTERED_RESEARCH_PACKAGEandPASS_RESEARCH_COMMIT_BYTES. Direct checks generate stdout receipts bound to all package-file hashes. No receipt is handwritten. Exact live source pins and historical task provenance are in SOURCES.json.Limitations
The proof assumes the retained complete Fourier and record-local product contracts, including their exact-grid/native/tape premises. The finite physical fixture uses the inherited direct/literal q2 child and reduced test guards. Production GL64/O73 execution, optimized full physical replay, Lean, Docker, the selected cold replay and the Gaussian/CRT/integer join were not run. No selected framework assumption is discharged. Independent human and formal proof-kernel review are unclaimed.
The raw Fourier convention is private to the complete convolution adapter. A standalone natural-order transform still requires the original Fourier API. The stated router count excludes routing internal to roots and the polynomial product.
Open next questions
Related open work
The completion refresh inspected the titles and bodies of all 15 other open account PRs created since the advance CLAIM: #6013 through #6023 and #6025 through #6028. None modifies the #3640/#3709 outer endpoint-reversal contract or computes a competing value for this fixture's three omitted outer routing calls. No additional scientific computations were run to compare those contributions. The advance CLAIM remains the canonical investigation pointer.
CI notes
Both hosted checks passed at exact head
d1f912d9c431414c7ea639baab0415c7183958d7in run 40592559504: plan in 4m32s and replay (research/reversal-free-convolution) in 7m29s. The completed checks were read once, and the run head matches the PR head.All local checks listed above also passed. Hosted execution certifies the bounded registered checks, including normal/optimized algebra and the physical fixture; it is not independent human or formal proof-kernel acceptance of the all-parameter refinement theorem. No CI failure remains.
Merge danger
Door: two-way. Blast radius: isolated research package.