Verify factor-four root-gap constant propagation - #31
Conversation
|
Focused verification status: PASS within the stated dependency boundary. The root-gap workflow completed successfully, including bytecode compilation, normal execution, execution under |
|
Independent companion PR #32 now supplies a stronger exact four-support entropy certificate. If both reviews are accepted, the combined explicit coefficient is |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3a4e41399e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| paths: | ||
| - "625/proofs/ROOT_GAP_CONSTANT_PROPAGATION.md" | ||
| - "625/audits/ROOT_GAP_CONSTANT_PROPAGATION_AUDIT.md" | ||
| - "625/experiments/root_gap_constant_supercheck.py" | ||
| - ".github/workflows/erdos625-root-gap-constant-supercheck.yml" |
There was a problem hiding this comment.
Include the canonical manuscript in path filters
When a PR only changes 625/proofs/COMPLETE_PROOF_SELF_CONTAINED.md, this workflow will not start because pull_request.paths is restricted to these four files, yet source_tag_check() explicitly reads that canonical manuscript and this proof note depends on those equation tags. That lets canonical equation-tag regressions merge without the new supercheck running; add the canonical source path, or broaden the filter to the relevant proof files.
Useful? React with 👍 / 👎.
| if not path.exists(): | ||
| return "SKIPPED (canonical source not present in working directory)" | ||
| text = path.read_text(encoding="utf-8") | ||
| for tag in ("\\tag{5.11}", "\\tag{5.13}", "\\tag{10.13}", "\\tag{11.1}"): |
There was a problem hiding this comment.
Check every canonical input tag
source_tag_check() says it guards the canonical equations used by the note, but this tuple omits several listed inputs, including the chromatic lower integer from (4.1), the entropy certificate (5.3), and the amplification definition (10.12). If any of those prerequisites are renumbered or removed in a PR that does run this workflow, the checker still prints PASS, so the regression gate does not actually cover the proof dependencies it claims to protect.
Useful? React with 👍 / 👎.
| if not path.exists(): | ||
| return "SKIPPED (canonical source not present in working directory)" |
There was a problem hiding this comment.
Fail when the canonical source is absent
If the canonical manuscript is renamed, deleted, or this checker is run in a checkout where it is missing, the verification still exits successfully and prints the overall PASS banner with only a SKIPPED note. Because this script is meant to guard the equations the new proof note depends on, absence of that source should be a hard failure rather than a successful supercheck.
Useful? React with 👍 / 👎.
Summary
This is a clean follow-up to PR #30, created directly from current
main. It isolates the arithmetic and asymptotic propagation from the manuscript's phase-resolved root displacement (5.11) to the final theorem constant.The central conclusion is that the midpoint construction retains
as the coefficient of
n/(log n)^3. Integer rounding costs onlyN+3, and the amplification theorem subtracts onlyo(n/N^3). Neither step causes an additional fixed halving.Conditional on the canonical inputs and their asserted phase-uniformity, the existing entropy certificate therefore supports
instead of the currently displayed
This is a factor-four constant propagation. No witness profile, overlap estimate, canonical manuscript, PDF, or Lean source is changed.
Files
625/proofs/ROOT_GAP_CONSTANT_PROPAGATION.md625/audits/ROOT_GAP_CONSTANT_PROPAGATION_AUDIT.md625/experiments/root_gap_constant_supercheck.py.github/workflows/erdos625-root-gap-constant-supercheck.ymlExact rounding lemma
For real
x>yandL>=0,Applied with
x=r_+,y=r_4^co, andL=log n, the additive loss is negligible becauseThe same note records a general placement inequality for
but non-midpoint placement remains experimental until all later constants are replayed.
Local verification
The standard-library checker currently reports:
It also:
Decimalarithmetic;python -Owith explicit exceptions rather thanassert;The phase scan suggests
min A(delta) approx 0.520701335491on a 1001-point grid, corresponding to an approximate phase coefficient0.031271565748. This value is not promoted as a theorem or replacement constant.Scope boundary
This PR does not independently verify:
o(1);Erdos625Statement.It verifies the deterministic rounding and coefficient propagation once those canonical inputs are accepted. The PR should remain a review artifact until the continuity/uniform-slack and event-intersection steps receive independent line-by-line review.