Repository navigation
Box ch. 7 content into numbered divs and callouts - #124
d-morrison wants to merge 9 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
State every condition in the identification results, add faithfulness where exchangeability is said to fail, separate the sign rule from the signed-diagram definition, reword passages close to the book, and add examples after definitions that lacked one. 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
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.
Also require the common cause's path to A to avoid Y. 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 Div balance holds (105/105), no duplicate Review: PR #124 — follow-up roundWhat changed since the last review roundPer the computed range
Both prior findings are resolved exactly as proposed, with no new issues introduced by this small, surgical diff. No other files changed in this range ( VerdictReady for merge — both outstanding findings from the prior round are fixed correctly and nothing new was introduced. Structured Review Data (JSON){
"schema_version": "1.1",
"reviewer": "claude",
"commit_sha": "1b64cbea0b9240577f994a77d47410335e2ad95c",
"verdict": "CLEAN",
"findings": [],
"detailed_assessment": "Verified chapters/07-confounding.qmd's two renamed box headings no longer duplicate their parent section headings and div balance remains 105 open/105 close.",
"holistic_assessment": "The 3-line follow-up commit fully addresses both prior-round findings without touching crossrefs, WORDLIST, or macro usage, so regression risk and review scope are both minimal."
}Reviewed commit: 76129fe |
|
Requested by Ezra · project thread
Part of #77
Before: ch. 7 (confounding) had 6 definitions, 1 theorem and 2 examples in numbered divs. Most of the chapter sat in plain prose, speaker notes, or Fine Point and Technical Point callouts. Some terms were used before they were defined, such as "confounding" itself and "no unmeasured confounding".
After: 65 numbered boxes. Each definition comes before its first use and is followed by a concrete example.
#thm-standardization-ch7, kept; its statement now lists its conditions).The Fine Point and Technical Point callouts now hold short summaries that point to the boxes after them, as in ch. 6 and ch. 12. No theorem div sits inside a callout or
.notes.How:
#def-backdoor-criterion-ch7,#def-backdoor-path-ch7,#thm-standardization-ch7,#def-sufficient-set,#def-swig, ...) and the section headings, because ch. 8 and ch. 12 link to them. All new ids are unique across the book.Checks:
quarto render chapters/07-confounding.qmdsucceeds with no warnings to html, docx and revealjs. The page has no unresolved?@references.@reference points forward, no id is duplicated across chapters, and the divs balance. The links into ch. 2, 4 and 6 point to ids or headings that exist.check-new-line-breaksandcheck-informal-definitionschecks pass.mmHgwas added toinst/WORDLIST.🤖 Generated with Claude Code
https://claude.ai/code/session_01Gh7L9vJhwrpvpBhzYMdjPK
Generated by Claude Code