[d-morrison]

Requested by Ezra · project thread

Bumps the latex-macros pin to 605844f, which includes macros#294: \sigmoid and \sigmoidf now print upright expit instead of σ, which also prints a standard deviation, a bandwidth and a singular value. The names stay as permanent aliases (macros#295), so no source change is needed. Pin-only change; not rendered locally.

---

Generated by Claude Code

[d-morrison]

/review

---

Posted by Claude Code (AI agent) --- not written by a human.
Generated by Claude Code

---

Generated by Claude Code

[github-actions]

Claude finished review — View run

---

The new pin SHA is the tip of main on d-morrison/macros, matching the submodule's tracked branch — confirmed.

Review: PR #1409 — Bump latex-macros so the logistic function prints as expit

Summary: A one-line submodule pointer bump (latex-macros: 16d943c7 → 605844f5), pulling in d-morrison/macros#294 and #295.

Verification performed:

• Confirmed 605844f586cfec66b5e278d6630bf22995c27cef is the actual tip of main in d-morrison/macros — not a fabricated or wrong SHA.
• Confirmed macros#294 does exactly what the PR body claims: \def\sigmoid{\sigma} → \def\sigmoid{\expit}, and \sigmoidf likewise now expands through \expit instead of \sigma\paren{...}. macros#295 adds a short rationale note in notation-choices.qmd that the aliases are being kept permanently.
• Confirmed \sigmoid/\sigmoidf are not used anywhere in rme's own .qmd content (only inside the vendored latex-macros/ submodule itself), so "no source change is needed" is accurate and this bump is a true no-op for currently-rendered pages.
• The commit range between the old and new pin also includes several unrelated upstream commits bundled along for the ride (Create classification appendix chapter with polished content #266 theme dropdown, read and incorporate "Martinussen T, Vansteelandt S, Andersen P. Subtleties in the interpretation of hazard contrasts" #287 exponential-family macros, read and incorporate "Royston P, Parmar M. Restricted mean survival time: an alternative to the hazard ratio for the design and analysis of randomized trials with a time-to-event outcome. BMC Med Res Methodol. 2013;152:152." #289 KL-divergence/surprisal macros, cover Aalen's additive hazard model #290 preimage macro, link to Statistical rethinking with brms, ggplot2, and the tidyverse #283/link to https://dev.peopleanalytics-regression-book.org/ #285 licensing). These only touch files outside what rme actually includes (rme only pulls in latex-macros/macros.qmd via {{< include >}}, never notation-choices.qmd, styles.css, etc.), and are purely additive to macros.qmd with no renamed/removed macros, so they pose no breakage risk to rme. Worth noting only as an FYI, not a blocker — pin bumps unavoidably sweep in whatever landed upstream since the last bump.
• CLAUDE.md's "no raw source change needed" claim checks out, and since no .qmd/.R file changed, the pre-commit render/lint/spellcheck checklist doesn't apply in the usual sense; CI (spellcheck, lint, link-checker, etc.) is green, with build and check-dois still pending at review time but unrelated to this change.
• No d-morrison/macros LaTeX macro definitions were added or changed in rme's own diff (the diff is submodule-pointer-only), so the macro-simplification/new-macro-candidate checks don't apply here.
• No AI-hallucination concerns: the PR numbers, commit SHA, and described behavior all check out against the actual upstream repo.

No issues found — this is a correct, minimal, well-described submodule bump with an accurate PR description, verified against the actual upstream commits and against rme's own usage of the affected macros.

Verdict

Ready for merge

Structured Review Data (JSON)

{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "d80da014579c0ca7981b308d763f34f2695d6e3d",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "The latex-macros pointer bump introduces no changes to rme's own qmd files, and sigmoid/sigmoidf are unused in rme content, so the new expit rendering is a verified no-op risk.",
  "holistic_assessment": "Scope stays narrow as a submodule pin bump, integration with rme's include-only-macros.qmd pattern is unaffected, and CI validation is green aside from unrelated still-pending jobs."
}

Reviewed commit: d80da01

[github-actions]

💰 Cost: $0.3891 (review) — run

[github-actions]

PR Preview Action v1.8.1
🚀 Preview available at: https://Morrison-Lab.github.io/rme/pr-preview/pr-1409/

_{Built to branch gh-pages at 2026-10-25 01:53 UTC.}