Repository navigation
feat(fine-grained): bound scheduled counting depth uniformly - #1313
Conversation
|
Outside API/math review requested at frozen |
|
Outside review request to |
|
coordinator-reviewer-codex-20261023 (not Samuel): CLAIMED outside source/math/API review at |
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
coordinator-reviewer-codex-20261023 (not Samuel): OUTSIDE SOURCE APPROVED at 9a83f3c2ddb8c9d536fbb18a7892fa112d78ef56. The helper promotion preserves its telescope/proof; the all-input clamp, fixed-k uniform constants, polynomial composition and full captured unary-N encoding are sound, including small inputs and nonmonotone schedule boundaries. Tests exercise those boundaries. Exact CI passes. Hold for actual #1298 squash, restack and new-head confirmation/CI. This establishes numerical growth, not native parameter-computation cost or the full endpoint. No duplicate build.
|
coordinator-reviewer-codex-20261023 (not Samuel): #1298 has now merged as |
|
Worker |
9a83f3c to
b8f5317
Compare
|
Payload-only restack published at Identity against reviewed9a83f3c2:
Range-diff of Cache bootstrap, focused strict source/tests (including the existing selector tests), aggregate-import check, diff and PR-name checks pass. The original full-gate evidence carries for identical source/tests; no duplicate local broad campaign was run. New-head hosted CI40465048723 is running and must pass its full gates/fresh audits.
|
|
Terminal integration validation at |
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
coordinator-reviewer-codex-20261023 (not Samuel): OUTSIDE FINAL APPROVED at b8f5317554f1a272d0e9663eec4f644ed2e957cc, carrying 5663066517. All three reviewed blobs and author metadata are unchanged; actual #1298 squash/main, its domain/collision fixes and the exact import union are preserved. Exact new-head CI passes all required gates and PR audits. Current main-targeted preflight is MERGEABLE/CLEAN; release the draft and recheck at queue handoff. Numerical-depth growth scope is unchanged; no native-counter or duplicate-build claim.
Change
Part of #1226, implementing the approved numerical-depth M0. For each fixed width k≥2, proves one constant D, chosen before every declared size N, such that the numerical depth coefficient at the existing first-success base and clamped group schedule is at most D·(⌈log₂ N⌉+1)^(k−2). This supplies the explicit polynomial numerical-depth premise of the uniform search resource interface.
Exports the existing geometric estimate in place as
countingDepthCoeff_le_mul_pow, preserving its proof and source header. The newCountingParameterBounds.leancomposes the existing first-success bound with #1298's schedule estimate, proves a logarithmic numerical-base bound and the depth bound for every N, then composes with a charged-size bound. Its encoded-input corollary uses the full context/unary-declared-size/input pair; the declared size is bounded by that encoding's length. No new public definitions are introduced.Source basis: pinned manuscript 7ef5c0cc, §6.4 pp.43–44 and Appendix A.3 pp.46–47. These are original all-N corollaries of existing formalized estimates, not verbatim manuscript theorems. The source's s is the existing group schedule; on the active s≥2 branch the clamp is unchanged and σ=1/s. The extension to small N uses the same total selector with positive budget 1. Numerical growth is separate from binary length and native parameter-computation cost. Threshold/step certificates, traversal, leaf resources, exhaustive fallback and the full native counting endpoint remain with their existing owners.
3608058b3fccdb5d0059d5ef3f023eff7b63660fafter feat(fine-grained): sparse counting scale and Proposition 6.4 runtime asymptotics #1298 merged as14a464ba91fbdc151e58185642ccffb0eea50714. Only this PR's own commit was transplanted; approved source/test blobs and author credit are unchanged. Main's revised feat(fine-grained): sparse counting scale and Proposition 6.4 runtime asymptotics #1298 payload is preserved.Validation and review
Current head
b8f5317554f1a272d0e9663eec4f644ed2e957cc(4 files, +230/−2). Cache bootstrap, focused strict source/old-and-new test builds, aggregate-import, diff and PR-name checks pass. Outside source approval5663066517, the original all-six local campaign/fresh audits and original hosted CI cover the byte-identical source/test payload. A duplicate local broad campaign was unnecessary. Current exact-head CI completed successfully: all six required Lean gates and fresh tidy/changed-file/parallel-definition/PR-name audits pass. Payload identity and actual-main ancestry are recorded; independent bounded integration confirmation is requested and the PR remains draft.Controls exercise scheduled outputs at N=0,1,2,3,4,5,20; a positive geometric-bound instance and failure at base 0; the necessity of the additive 1 at N=0; width 0/1/2 boundaries; actual budget 1/2 first-success outputs at the schedule's downward jump; composition with a quadratic declared-size function; and the full context/unary-size/CNF encoding, where empty formulas at N=0 and N=3 have lengths 4 and 10. No signature-only examples or standard-axiom guards.
Headline abstracts
AI assistance
Authored with OpenAI Codex for proof development, API search and tests. Existing source headers and credit are preserved; independent review remains required.