Skip to content

Bump latex-macros so the logistic function prints as expit - #259

Merged
d-morrison merged 1 commit into
mainfrom
claude/project-thread-gz6w0h
Oct 25, 2026
Merged

d-morrison merged 1 commit into
mainfrom
claude/project-thread-gz6w0h

Conversation

@d-morrison

Copy link
Copy Markdown
Contributor

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

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DcoxKs4KBbN2HfeyoeUw5h
@d-morrison d-morrison self-assigned this Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

/review


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


Generated by Claude Code

@github-actions

Copy link
Copy Markdown
Contributor

Claude finished review — View run


I found no issues in this PR. It is a one-line latex-macros submodule bump from 16d943c to 605844f, and nothing else changes. No .qmd, R code, workflow or references.bib file is touched, and no math is added or changed, so I skipped the macro check.

The new submodule pin resolves in the local checkout. Its HEAD is 605844f, "Record that the sigmoid aliases are permanent (#295)". I did not diff the macro changes between the two commits, so I haven't confirmed that no macro pds uses was renamed or removed. That risk is small, since the commit message suggests aliases are being kept.

This is a dependency bump of the kind Dependabot makes, which CLAUDE.md allows. It stays within the PR scope rules.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "bb600d048c05670af9cc8e6afe896784dce64285",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "The only changed path is the latex-macros gitlink, bumped to 605844f, which resolves in the local checkout; no content, code, or config changes exist to fail.",
  "holistic_assessment": "A routine submodule pointer bump within scope, with no math or prose changes; I did not diff the macros between the two pins, so removed-macro breakage is unchecked but unlikely."
}

Reviewed commit: bb600d0

@github-actions

Copy link
Copy Markdown
Contributor

💰 Cost: $0.1491 (review) — run

@github-actions

github-actions Bot commented Oct 25, 2026 •

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

2026-10-25 03:19 UTC

@d-morrison
d-morrison merged commit 398bb18 into main Oct 25, 2026
27 checks passed
@d-morrison
d-morrison deleted the claude/project-thread-gz6w0h branch October 25, 2026 03:18
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