Add predictor selection chapter (RMB Ch10) with HERS and WCGS DAG examples#400
Conversation
Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/bdf1cb6e-d241-4d68-b607-9e8a9c8b277f Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
… caption Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/bdf1cb6e-d241-4d68-b607-9e8a9c8b277f Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/bdf1cb6e-d241-4d68-b607-9e8a9c8b277f Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
…::snapshot()) Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/b2029468-74e3-4821-9460-ffbaa54eaa58 Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Fixed in commit |
There was a problem hiding this comment.
Pull request overview
Adds a new “Predictor Selection” chapter (RMB Ch10) to the Quarto book/website, including an HERS DAG example and accompanying sections.
Changes:
- Adds
dagittyto project dependencies (DESCRIPTIONSuggests +renv.lock). - Introduces new chapter/section QMD files for predictor selection overview and HERS DAG-based discussion.
- Wires the new chapter into the Quarto book + website nav, and updates the spelling WORDLIST.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
DESCRIPTION |
Adds dagitty to Suggests so renders can use it. |
renv.lock |
Locks dagitty version for reproducible installs. |
predictor-selection.qmd |
New top-level chapter file that includes the new sections and examples. |
_sec-pred-sel-overview.qmd |
New introductory overview of predictor selection strategies. |
_sec-pred-sel-dag.qmd |
New DAG theory + HERS DAG section and LDL modeling workflow. |
_quarto-book.yml |
Adds the new chapter to the book build. |
_quarto-website.yml |
Adds the new chapter to website render list + navbar. |
inst/WORDLIST |
Adds new technical terms for spellchecking. |
|
@copilot apply changes based on the comments in this thread |
…r, WORDLIST cleanup Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/f25e1fc0-22c8-49b7-9dd4-6d47d77029a6 Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Applied all review changes in
|
Resolved conflicts in: - .github/copilot-instructions.md: kept both 'Figures and Tables: Use Div Format' (our branch) and 'Landscape Tables in PDF' (main) sections - inst/WORDLIST: merged union of both word lists, removing nested conflict markers Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Resolved in
|
…lout title fixes, attribution note Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/3baf0a50-ab98-4981-87b5-cf7cbd407687 Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/3baf0a50-ab98-4981-87b5-cf7cbd407687 Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Added in
Added in
Also fixed the callout heading levels ( |
…pdate copilot-instructions Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/d95a3882-3301-4159-87a9-55279a38ce57 Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
…g.qmd Agent-Logs-Url: https://github.com/d-morrison/rme/sessions/795cb438-f0b3-4341-84ba-d325849a080d Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Fixed in |
2026-04-16 22:03 UTC |
|
correct the statement of the main goal of the HERS study: it was estimate the causal effect of HT on CHD event risk. define a confounder as a common cause of an exposure and outcome. |
_sec_hers_intro.qmdto_subfiles/shared/_sec_hers_intro.qmdin_sec-pred-sel-dag.qmd