[d-morrison]

Fixes #245.

A definition div holds the definition, notation and display equation; commentary goes in a #rem- div (Morrison-Lab/sds#444 (comment)).

• #def-squared-error-risk loses its closing sentence ("It is the risk under squared error loss").
• The sentence moves to #rem-squared-error-risk-is-risk, in its own subfile, included after the definition and its example (they share one file, so the remark cannot sit between them without splitting it).
• The definition's anchor is unchanged.

A scan of the other _def-* files in pds found no further commentary.

Checked locally: check-term-links is clean. No local render; please check the PR preview.

Owning thread: Bias is prediction error. Companion fixes: Morrison-Lab/sds#454 (merged), https://github.com/Morrison-Lab/lds/pull/1344 (merged).

---

Generated by Claude Code

[github-actions]

Claude finished review — View run

---

The change is small and I found nothing to fix. It moves the one-line "it is the risk under squared error loss" sentence out of the #def-squared-error-risk div. It now sits in a new one-div subfile that the page includes straight after the definition.

Checks

• One div per subfile: _subfiles/_rem-squared-error-risk-is-risk.qmd holds a single top-level #rem- div, so it meets the one-div-per-subfile rule. The rem- id prefix is the one the gha checks expect for remarks.
• One term per definition div: _def-squared-error-risk.qmd now defines only the squared-error risk, with the cross-reference sentence gone.
• Links and forward references: #def-risk and #def-squared-error-loss are both defined earlier in expectation.qmd (_def-risk.qmd and _def-loss-function.qmd), so there are no forward references. #def-squared-error-risk is defined immediately above the remark.
• Include and anchors: the include at expectation.qmd:118 uses the repo's pds/_subfiles/... path convention and is followed by the usual slidebreak. #def-squared-error-risk keeps its id, so no external link breaks.
• Style: the remark has no math, so the macro check does not apply. The .remark class follows existing usage in _def-pdf-pmf-cdf.qmd. The older _rem-risk-loss-exercise.qmd has no class, which this PR does not touch.
• Build and CI: there are no changes to _site/, _freeze/, bibliography, workflows, or R code. I did not render the page or run any CI checks.

Prior reviews of this PR: none were available to me, so there is nothing earlier to confirm.

Verdict

Ready for merge

Structured Review Data (JSON)

{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "016e30f465f9a58530574091838093c4ce4cd6c8",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "Traced the new _rem-squared-error-risk-is-risk.qmd subfile and its include in expectation.qmd; the #def-risk and #def-squared-error-loss targets are defined earlier, so there are no forward references or broken anchors.",
  "holistic_assessment": "The change splits one remark out of a definition div to satisfy the one-term-per-div and one-div-per-subfile rules; scope is minimal, no generated files or CI config are touched, and no regression risk was found."
}

Reviewed commit: 016e30f

[github-actions]

💰 Cost: $0.1575 (review) — run

[github-actions]

PR Preview Action v1.8.1
Preview removed because the pull request was closed.

_{2026-10-23 17:51 UTC}