Repository navigation
Box ch. 2 content into numbered divs and callouts - #123
d-morrison wants to merge 7 commits into
Conversation
Part of #77. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
…ed paraphrases Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
|
Claude finished review — View run The title change accurately reflects the content (plugging the random variables I checked whether other remark/div titles in the file still contain inline math (which would risk the same PDF build failure), and found none in this chapter's titles — this was the only offending heading. No other part of the diff changed since the prior round ( VerdictReady for merge — the only change since the previously reviewed commit ( Structured Review Data (JSON){
"schema_version": "1.1",
"reviewer": "claude",
"commit_sha": "e4e5296a6c225a860810dde1fb29cdbbb8ce047b",
"verdict": "CLEAN",
"findings": [],
"detailed_assessment": "The only change since the last reviewed commit is a one-line heading edit in chapters/02-randomized-experiments.qmd removing inline math from the rem-ip-weight-notation title to fix a PDF build failure, and the replacement wording remains accurate to the surrounding proof text.",
"holistic_assessment": "This single-line title fix introduces no cross-reference, macro, or scope regressions, and the previously reviewed structural, mathematical, and lab-manual compliance checks across the rest of the chapter remain valid and unaffected."
}Reviewed commit: 4089220 |
|
Requested by Ezra · project thread
Part of #77
Before: ch. 2 had 5 definition divs and no numbered results, examples or remarks. Its Technical Points 2.1-2.3 and Fine Points 2.1-2.2 were callouts that mixed definitions, results and proofs, so none of them could be cross-referenced.
After: 17 definitions, 13 propositions (each with a proof), 27 examples and 4 remarks, plus 5 warning callouts, 2 tip callouts and 1 note callout.
def-exchangeability,def-conditional-exchangeability,def-identification,def-standardization,def-ip-weights) are kept, because ch. 12 and ch. 17 link to them. New ids that could collide take a-ch2suffix (def-stratification-ch2,def-positivity-ch2,def-pseudo-population-ch2,prp-pseudo-population-ch2).def-ip-weightsstill duplicates ch. 12's id until Fix duplicate crossref id #def-ip-weights (chapter 12) #114 merges.How:
.notesblock or another div.Checks:
quarto render chapters/02-randomized-experiments.qmdsucceeds forhtml,docxandrevealjs, with no warnings and no unresolved?@references.--to latexalso succeeds. The PDF could not be compiled locally because TinyTeX is not installed in this container.spellingfinds no new words beyond those already flagged on main, which come from math macros and names.🤖 Generated with Claude Code
https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
Generated by Claude Code