Repository navigation
Save eight coefficient ADDs in the actual Gaussian reader (no new κ) - #5919
Merged
Mister-Meeseeks merged 1 commit intoOct 27, 2026
Merged
Mister-Meeseeks merged 1 commit into
Mister-Meeseeks merged 1 commit into
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
An explicit rational centre-basis shear reduces the actual rank-30 Gaussian reader's four-shear coefficient tariff by eight ADDs:
After Ali Romman's #5646 basis, use
D5 += (3/40) D4andJ4 -= (3/40) J5. The source/reader product is unchanged, every arbitrary dirty centre is restored, and the word still uses four shears and thirty borrowed fields. This is a finite explicit rational-kernel construction with a scoped algebraic proof. No new kappa or selected-frontier change.The common denominator changes from 960 to 1200, adding a second factor of five. The maximum coefficient remains one, and the maximum absolute row sums remain 9/4 and 1036/15. This improves the declared coefficient-support tariff; physical rational arithmetic may have a different price.
Only
research/rational-reader-basisis added. The first strict improvement ends the search; no optimality is claimed. The result was produced by Nick Petalas. Credit Ali Romman #5646 for the actual factors, published basis and open question, David Ganz / Dugongue #5634 for the reader identity, #5640 for the grouped tariff, and the retained centre-code-services lineage and source notices.Evidence
dfba21817680dbbd1ca0455cae5e5e93c97710fb. Official export/input coverage, a read-only Git-free optimized replay and source-corruption rejection pass.From the repository root:
Expected statuses are
PASS_RATIONAL_READER_BASIS,PASS_REGISTERED_RESEARCH_PACKAGEandPASS_RESEARCH_COMMIT_BYTES. The verifier emits a generated hash-bound JSON receipt. No predecessor verifier, rank/inverse audit, local-optimum replay or historical suite is run.Limitations and unrun obligations
The supplied boundary
DJ=0and factor construction are cited from the predecessors. The new proof establishes preservation and the new support count; it does not independently validate those predecessors. Rational arithmetic interiors, rounding, routing, tape/storage, a complete recursive invoice, native/outer admission and a full multiplication theorem remain open. No proof assistant or selected cold replay was run. All five framework assumptions remain. Independent human and formal proof-kernel review are unclaimed.Open next questions
Related open work
Refreshed this account's open PRs created after the advance CLAIM: #5920, #5921, #5922, #5923, #5924, #5925, #5926, #5927 and #5928. Their titles and result scopes were read; none studies the actual rank-30 Gaussian-reader coefficient bill. No additional computation was run to compare other contributions.
No open PR studying this same reader was found in the earlier live searches for
5634,5646andGaussian reader. Merged #5646, #5634 and #5640 are credited predecessors. The other rank-30 search reference concerns a historical moment profile, not this reader. These searches do not cover unpublished work.CI notes
At
dfba21817680dbbd1ca0455cae5e5e93c97710fb, run 40513266276 finished with plan failed; replay skipped. The only failed step is Onboarding helper tests. The unchangedtests/test_research_current_assignment.pyends withFAILED (failures=8, errors=2, skipped=22). Its two errors areKeyError: 'follow_up_steps'andKeyError: 'catalog_first_step'; the eight failures concern current-assignment card/receipt and missing/sparse-reference expectations.The same step, test file and failure summary were directly confirmed for PR #5918 at
d3d1e2155620e2a491523c759e1569587678b922in run 40506994991. This package adds only its seven scoped files and changes none of those shared tests or helpers. Maintainers own the shared failure; no repair or rerun was attempted here.Hosted replay has not certified this package. Only the local scientific and publication checks listed above have run. CI did not execute the package replay.
Merge danger
Door: two-way. Blast radius: isolated research package. Existing sources, shared tooling and the selected frontier are unchanged.