Skip to content

Bump latex-macros so the noise prints as ν - #192

Merged
d-morrison merged 1 commit into
mainfrom
claude/mu-true-mean-fn-fs0x15
Oct 16, 2026
Merged

d-morrison merged 1 commit into
mainfrom
claude/mu-true-mean-fn-fs0x15

Conversation

@d-morrison

Copy link
Copy Markdown
Contributor

Requested by Ezra · project thread

Before: the noise macro \noise printed ε in the noise exercises and theorems.

After: the latex-macros pin moves to the merge commit of macros#175, where \noise is an alias of the deviation from the population mean and prints ν (Ezra, 2026-10-16). No source file changes; the pages using \noise (the noise second-moment theorem and exercise, and others) print ν.

How: submodule bump only. variance-covariance.qmd renders to HTML. Heads-up: ν also prints for degrees of freedom in the macros.

🤖 Generated with Claude Code

https://claude.ai/code/session_01H8gaSFRnPKEfpCbBRqEzZH


Generated by Claude Code

@d-morrison d-morrison self-assigned this Oct 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Claude finished review — View run


Submodule bump only (Dependabot-style). Nothing to verify further locally without the upstream repo.

Code review

The PR changes one thing: the latex-macros submodule pointer moves from e7a600ec to a9d2669f. This is the routine macro-submodule update that CLAUDE.md says Dependabot normally handles. It touches no .qmd, R, config or workflow files, and it commits no build artifacts. I didn't inspect the upstream macro changes, so I can't say whether any macro was renamed or removed in a way that would break pages that use it. A full site render in CI would catch that.

No issues found. Checked for bugs and CLAUDE.md compliance.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "af60d081524af2b8ed54897891df362c45c596d3",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "The only changed path is the latex-macros submodule pointer, e7a600ec to a9d2669f; there is no parsing, logic, or link surface to fail in this diff.",
  "holistic_assessment": "Scope is a single routine submodule bump; the remaining risk is a renamed upstream macro breaking a page, which the CI render would catch."
}

Reviewed commit: af60d08

@github-actions

Copy link
Copy Markdown
Contributor

💰 Cost: $0.1285 (review) — run

@github-actions

github-actions Bot commented Oct 16, 2026 •

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

2026-10-16 22:49 UTC

@d-morrison
d-morrison merged commit 3ff4661 into main Oct 16, 2026
21 checks passed
@d-morrison
d-morrison deleted the claude/mu-true-mean-fn-fs0x15 branch October 16, 2026 22:49
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