Repository navigation
Tooling: 1.54× faster production data_corners.cpp via cached update columns with identical receipts - #2320
Merged
Mister-Meeseeks merged 2 commits intoOct 15, 2026
Conversation
…d-fixed Wire the proved per-pivot cached-column loop from research/data-corner-performance/ into research/copied-fixed/data_corners.cpp. Update the certificate local_sha256 pin only for that file. Demote the pairwise-era live pin in the opt-in package by vendoring baseline.cpp. Prepared by Wentao Li with substantial AI assistance.
Record the pinned-CPU five-pair benchmark medians and update the package README/TASK evidence strings. Prepared by Wentao Li with substantial AI assistance.
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.
Change and scope
At base
0fdd7d82de03e43c9e5a1000dfbcd380d1b687d2, productionresearch/copied-fixed/data_corners.cppstill paid a fresh update-column rebuild per pivot. The opt-in adapterresearch/data-corner-performance/already proved a per-pivot cached-column loop with identical 4,073,300-pair receipts. This PR wires that equivalent loop into the production source.Only the
local_sha256pin forresearch/copied-fixed/data_corners.cppchanges insideresearch/copied-fixed/certificate.json. The opt-in package vendors pairwise-era bytes asbaseline.cppand records the pre-production digest underprovenance_sha256, so research-packages CI does not fail working-source drift. Historical inventories without VERIFY matrices are left as provenance.Tooling improvement; no new kappa. Selected frontier untouched. Does not edit the e8-bit-r420 MANIFEST tree (#2263/#2276).
Result
On Linux / CPython 3.12.3 with
taskset -c 0, five alternating fresh-process pairs gave median wall time 170.101 → 110.463 s (1.54×). Timing is host-descriptive and is not a correctness gate. Complete receipts match the baseline checker (4,073,300 pairs; 4,073,290 good / 10 fallback).Package:
research/data-corner-production/.Tip:
2c7dfd60f7a45d7c6b433afb9d73346f13c59832.Validation
Expected:
Default verify is read-only. Temp-copy column-fill foil rejects via subprocess failure. Package gates under
-Ostill launch asserting C++ children (NDEBUGrejected). Fullmake verify, selected cold replay, Lean, and multiplication-theorem obligations were not rerun for this isolated checker change.Overlap and credit
Closest priors (different delivery):
research/data-corner-performance/(opt-in cached columns; this is the production wiring), CrocSwap/integer-mult-bounds#756, CrocSwap/integer-mult-bounds#2257 / #2263 / #2276 (same production-promotion + provenance pattern). No competing open PR on productiondata_corners.cppat refresh.Credit the authors of
research/data-corner-performance/and the copied-fixed lineage. Prepared by Wentao Li (VictorLiwentao, @Is__Victor) with substantial AI assistance, under Apache-2.0.