Repository navigation
Nonauthor AMEND review of main PR125 shrinking-witness bound - #72
Conversation
The divided-difference Jacobian, covariance floor, determinant soft factors, cubic height gap, and separation integral support the uniform O(k r^3) expectation. The probability quantifiers and the second-window transition need a narrow repair. Scientific effect is none. Co-authored-by: Dylan Roy <d6g8k5htny@privaterelay.appleid.com>
|
To the author of Math- #72 (head What I read and ran. The full three-dot diff; Findings.
Scope and status. No status path is touched ( Verdict: AMEND (items 2 and 4 are required; 1 and 3 are wording). The record's dispositions, five ACCEPT and one AMEND, are sound at the stated scope, and the expectation bound Correction, 13 October 2026, from a recheck of this comment. Same session, same provider: the recheck was run by this session and a same-provider helper, so it carries no independence. Every rational and line reference above was recomputed or reread at the reviewed commit. The verdict stands. These sentences were wrong or incomplete:
Generated by Claude Code |
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
Unresolved moderate mathematical corrections remain in the review record.
Review effort: Lite
Findings: None
What changed in this PR
This PR records a scoped nonauthor AMEND review of PR125’s shrinking-witness bound and adds exact arithmetic certificates.
Changes:
- Documents review scope, amendments, and accepted interfaces.
- Adds rational checks for supporting identities.
- Records unresolved review corrections.
| File | Summary |
|---|---|
reviews/pr125_shrinking_witness_nonauthor_20260926/REVIEW.md |
Review findings, scope, and amended bounds. |
reviews/pr125_shrinking_witness_nonauthor_20260926/algebra_check.py |
Exact verification of algebraic identities. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Dylan Roy <d6g8k5htny@privaterelay.appleid.com>
Scientific effect: NONE. This review does not edit main pull request 125, and it does not infer pin, intermediate-annulus, or global RN closure.
Verdict
AMEND
docs/rn_d5_shrinking_witness_20260926.mdat00c9d08a146c60cfc026f2167c5a1d9e5678e34d(blobd29d00f205f12ee9619651165ee588422d777fd4).The expectation bound stands:
E[T_{r,j,2}(η)] ≤ C k r^3for ordered index-
jpairs inD_ρwith separation in[η, δ_*].Accepted interfaces
δ^d, and the density identityp(∇f(x),∇f(y)=0) = δ^{-d} p(G_0,G_1=0).(G_0, G_1), uniform for smallδ_*andr.|det|isO(δ), and the product isO(δ^2).-μ δ^3/12 - ν δ^4/24 - ρ δ^5/80.∫ s ds, finite asη ↓ 0in everyd ≥ 2.Required repair
The printed Markov event says “separation at least
η”, whileTonly integrates[η, δ_*]. Pairs farther thanδ_*areO((k r^3)^2)by the fixed-separation formula (15) and are absorbed for smallr. The second height window is lost onδ = O((k r^3)^{1/3}), not on every pair merely becauseηmay shrink. The positive limit of∫ s dsis the deleted-indicator majorant, not a sharpΘ(k r^3)coefficient.Record:
reviews/pr125_shrinking_witness_nonauthor_20260926/REVIEW.md.python3 -B -S reviews/pr125_shrinking_witness_nonauthor_20260926/algebra_check.pyprintsall certificates passed.