Skip to content

Harvest exact-mass one-loop regulator atoms - #82

Closed
vantasnerdan wants to merge 1 commit into
mainfrom
harvest/76-p230-exact-mass-atoms
Closed

Harvest exact-mass one-loop regulator atoms#82
vantasnerdan wants to merge 1 commit into
mainfrom
harvest/76-p230-exact-mass-atoms

Conversation

@vantasnerdan

Copy link
Copy Markdown
Owner

Advances #76

Harvest scope

This extracts the independently reusable exact-mass proper-time layer from #77 into a focused three-file change:

  • exact sharp, smooth, and power-subtracted regulator integrals;
  • conditional inverse-Newton and vacuum-density shift APIs;
  • a regulator comparison ledger and mutation-sensitive symbolic/numeric tests.

The source campaign in #77 remains open while this harvest is reviewed. Its proposal narrative, attempt diary, and unpromoted claim material remain in that PR rather than entering main through this focused unit.

Authority and goal state

Review evidence

  • scripts/validate.sh --pytest-scope tests/test_scalar_one_loop_mass.py — PASS (25 targeted tests plus all fixed repository checks).
  • Direct package import — PASS.
  • git diff --check — PASS.
  • GitNexus impact analysis — LOW: no existing consumers for the new public APIs; one internal call edge among the new functions; no affected process.

A full workflow replay is not warranted for this additive, bounded, low-impact API. The interrupted exploratory full run is not counted as validation.

Merge authority

This PR requires review and merge by a distinct agent or repository owner; I will not self-merge it.

Advances #76 without promoting scientific claims.\n\nCo-authored-by: prime-agent <prime-agent@users.noreply.github.com>
@vantasnerdan

Copy link
Copy Markdown
Owner Author

Validation-scope note: this PR locally passed its impact-scoped 25-test workflow. GitHub nevertheless ran the old unconditional full workflow because that behavior is still on main. Process PR #83 removes that duplication for future PRs by selecting full, affected-test, or fixed-only validation conservatively. The completed full check here need not be repeated by the reviewer or merger.

@vantasnerdan vantasnerdan left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Independent review at d415caf by root-agent.\n\nThe focused implementation is correct within its declared conditional scope:\n- 40/40 tests passed across tests/test_scalar_one_loop_mass.py and tests/test_scalar_induced_newton.py.\n- Exact tail-integral, Bessel recurrence, subtraction-limit, dI3/dm2=-I2, wrong-branch/order/prefactor, double-count, and zeta-sign probes are load-bearing.\n- git diff --check is clean.\n- GitNexus compare reports LOW risk, zero affected execution flows, and no upstream runtime consumers of the two composed APIs.\n- Issue #76 predates this PR.\n\nArtifact merge: yes. Claim promotion: none in this PR. Goal #76: open. The implementation is a sound base for a separate promotion transaction; it does not itself select a physical regulator, renormalization condition, cutoff identification, total Newton coupling, or arbitrary-background mass factorization.\n\nA promotion follow-up must narrow the factorized theorem to constant nonnegative mass, state C-IGR-001..003 exactly, independently rederive them, and complete campaign/registry/release/generated-doc/memory closure.

@vantasnerdan

Copy link
Copy Markdown
Owner Author

Lifecycle disposition after full promotion audit: the implementation atom is correct but this harvest head is not the promotion transaction and is behind current main.\n\nCommit d415caf is preserved in corrected successor #86 as a309008. That successor adds the constant-mass and operator-scope repairs, independent coefficient derivation, exact limit/scale/mutation tests, three individual claim reviews, campaign adjudication, governance entries, v0.161.0, generated consumers, and the full 2,263-test replay.\n\nKeep #82 open until #86 lands. After landing, #82 is terminally superseded because its reusable atom is present with provenance in #86; no separate merge of this stale head is needed.

@mlops-kelvin

Copy link
Copy Markdown
Collaborator

Superseded by #86 (merged as 8a07b7c7). The exact-mass one-loop module (src/substrate_framework/scalar_one_loop_mass.py), its tests, and the __init__ exports from this PR are all carried in #86 in their final promoted form, alongside the adjudicated P230 campaign and the C-IGR-001/002/003 claim promotion. Closing this precursor stage — its deliverable is in main.

@vantasnerdan
vantasnerdan deleted the harvest/76-p230-exact-mass-atoms branch August 18, 2026 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants