Skip to content

Justify sum steps as distributing the sum over the terms - #243

Merged
d-morrison merged 1 commit into
mainfrom
claude/distribute-sum-wording
Oct 23, 2026
Merged

d-morrison merged 1 commit into
mainfrom
claude/distribute-sum-wording

Conversation

@d-morrison

Copy link
Copy Markdown
Contributor

Applies the rule in Morrison-Lab/psw#335 (Ezra: "split is informal ... distribute is better") to the five "split the sum" uses. Wording only; no equation changes.


Generated by Claude Code

Applies the rule in Morrison-Lab/psw#335.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PjFUktL4yzBtLbYYmZhoQL
@github-actions

Copy link
Copy Markdown
Contributor

Claude finished review — View run


Code review

I found no blocking issues in PR #243 at 8f30e9e3a99df66ae356ebe678aa54cebcf54cfb.

The PR changes five annotation or prose phrases across five _subfiles/ fragments. Each one swaps "split the sum" for "distribute the sum over the terms":

  • _subfiles/_def-cond-expectation.qmd:680

  • _subfiles/_exr-fubini-joint-disc-infinite.qmd:70

  • _subfiles/_exr-fubini-joint-mixed-infinite.qmd:49

  • _subfiles/_sol-linearity-use.qmd:3

  • _subfiles/_thm-lotus.qmd:133

  • Math and macros: The diff changes only \text{...} annotations and one prose sentence. It adds no new math symbols, so there is nothing to check against the macros.

  • Anchors and cross-references: No #id anchors, cross-references, chunks, definition divs or paths change. The @thm-linearity-expectation reference in _sol-linearity-use.qmd is untouched.

  • Consistency: A grep for the old wording finds no remaining "split the sum" in *.qmd files under _subfiles/ or the top level. The two "split the integral at 0" phrases in _subfiles/_def-cuhaz.qmd:23 and :80 describe a different operation (splitting the domain), so leaving them alone is correct.

  • Build hygiene: The PR changes no generated files, bibliography entries, spelling wordlist entries, or workflows.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "8f30e9e3a99df66ae356ebe678aa54cebcf54cfb",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "All five changed fragments only reword a text annotation or sentence from 'split the sum' to 'distribute the sum over the terms'; no math, anchors, or cross-references are altered and no stale occurrences remain.",
  "holistic_assessment": "The change is a small, well-scoped wording consistency edit with no integration, regression, build, or macro-usage concerns."
}

Reviewed commit: 8f30e9e

@github-actions

Copy link
Copy Markdown
Contributor

💰 Cost: $0.1634 (review) — run

@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.

2026-10-23 07:09 UTC

@d-morrison
d-morrison merged commit 4f969d0 into main Oct 23, 2026
29 checks passed
@d-morrison
d-morrison deleted the claude/distribute-sum-wording branch October 23, 2026 07:08
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.

2 participants