Repository navigation
Box ch. 9 content into numbered divs and callouts - #128
d-morrison wants to merge 6 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
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.
…grammar Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
|
Response to the Claude review of 13054b9, fixed in 01337f8:
Posted by Claude Code (AI agent) --- not written by a human. Generated by Claude Code |
|
Claude finished review — View run Both prior findings are correctly addressed. The three conditions in the new I checked the rest of the diff for new issues: the only other change in this range is this same hunk (the No new findings. Both issues raised in the prior review (commit VerdictReady for merge — both outstanding issues from the prior review round are fixed correctly and verified against the Chapter 6 source; no new findings. Structured Review Data (JSON){
"schema_version": "1.1",
"reviewer": "claude",
"commit_sha": "67e2ff87a1df41926156e1d9ffb94d52deab547f",
"verdict": "CLEAN",
"findings": [],
"detailed_assessment": "chapters/09-measurement-bias.qmd def-causal-dag-strict now quotes the three Technical Point 6.1 conditions verbatim and separates the causal Markov assumption as a linking statement, fixing the prior mismatch.",
"holistic_assessment": "The grammar fix and definitional restatement resolve both prior blocking findings without touching crossrefs, WORDLIST, or macro usage, so scope and regression risk remain low."
}Reviewed commit: 01337f8 |
|
Requested by Ezra · project thread
Part of #77
Before: ch. 9 (measurement bias and "noncausal" diagrams) had 3 definitions and 1 example in numbered divs. The rest of the content sat in plain prose, in speaker notes, or inside the Fine Point and Technical Point callouts, where nothing could be cross-referenced. "Measurement bias" was bolded in the intro before it was defined.
After: 50 numbered boxes:
8 definitions. New ones:
def-measurement-biasnow also covers mismeasured confounders, as Section 9.3 uses the term.def-causal-dag-strictnow restates the three Chapter 6 conditions (Technical Point 6.1) and the causal Markov assumption it builds on.5 propositions, each with a proof:
23 examples. These include a worked version of the pedestrian experiment with hypothetical numbers (ours, marked). They also include edge-by-edge readings of Figures 9.9, 9.10, 9.12 and 9.15–9.19, checked against the book's figures.
14 remarks. One, our own, explains when the graphical error classification and the error-variable definitions of Technical Point 9.1 agree. An example shows a misclassified binary outcome whose error is differential under the Technical Point definition, even though the recorder's lapses ignore treatment.
14 callouts: 7 notes (Technical Point 9.1 and Fine Points 9.1–9.6), 4 warnings and 3 tips. All Fine and Technical Points already had callouts, and Add callouts for Fine and Technical Points mentioned without one #122 added none for ch. 9. As in the ch. 7 PR (Box ch. 7 content into numbered divs and callouts #124), each one now holds a short summary that points to the boxes after it, and the titles are unchanged.
How:
def-measurement-error,def-measurement-bias,exm-hepatitis,def-causal-dag-strict) and every##heading is kept. Ch. 20 links to the 9.3 heading anchor. One###heading lost its inline$H$and is now "Hidden Factors Behind Obesity". Nothing links to it.07-confounding.qmd#def-backdoor-criterion-ch7,#thm-standardization-ch7,03-observational-studies.qmd#def-endeand06-graphical-representation.qmd#def-causal-markov..notes. No div title has inline math, and no@refpoints forward or sits inside math.pdftotextof the book chapter finds only box titles, section titles and math symbol runs. Several runs that were already on main are reworded too.misclassifiedis added toinst/WORDLIST.Checks:
quarto render chapters/09-measurement-bias.qmdto html, docx and revealjs succeeds with no warnings, and there are no unresolved?@refs.--to latexalso succeeds. I could not compile the PDF here because TeX is not installed.check-new-line-breaksandcheck-informal-definitionschecks pass.spellingfinds nothing new beyond the words already flagged on main.🤖 Generated with Claude Code
https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK