-
Notifications
You must be signed in to change notification settings - Fork 14
Resolve merge conflicts, refine WCGS DAG assumptions/layout, and expand predictor-selection methods, citations, and documentation guidance #552
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
bf8a541
Initial plan
Copilot a6ec77a
fix: clarify HERS causal goal and confounder definition in predictor …
Copilot e036c71
Merge main into branch and update WCGS DAG causal framing
Copilot b464427
Refine WCGS DAG assumptions and simplify confounder wording
Copilot 668a2cb
Add prediction metrics definitions and overfitting example
Copilot 2adf20d
Refine overfitting example code clarity in prediction section
Copilot f5e3326
Add iterative 10 percent rule BWE workflow for WCGS confounders
Copilot 590bb55
Refine BWE section formatting and citation detail
Copilot 2055aeb
Refactor BWE code clarity in WCGS 10 percent section
Copilot 5efc92b
Address review thread citations and shared HERS intro consolidation
Copilot ece4666
Polish 10 percent citations and remove duplicated HERS count
Copilot 4ea0880
Add transition text after shared HERS intro include
Copilot 2e24a25
Refine HERS intro phrasing and DAG transition wording
Copilot cdd6d8c
Restore landscape-table guidance and correct HERS objective wording
Copilot 7b61dde
Update WCGS DAG naming/layout and add HERS/WCGS data dictionaries
Copilot bf2bd50
Fix data-dictionary caption placement in predictor DAG section
Copilot cd70aee
Update _sec-pred-sel-dag.qmd
d-morrison 4f8724f
Apply review-thread updates for predictor selection and citations
Copilot a0ff2b8
Address remaining review nits for HERS citations and WCGS formula rob…
Copilot 4126e84
Merge branch 'main' into copilot/edit-predictor-selection-chapter
d-morrison ead43bf
Merge branch 'main' into copilot/edit-predictor-selection-chapter
d-morrison 6780abf
Merge origin/main and resolve conflicts preferring branch content
Copilot c0c2275
Merge latest origin/main and resolve conflicts
Copilot e7927d3
Merge branch 'main' into copilot/edit-predictor-selection-chapter
d-morrison 971da91
Apply review-thread citation and formatting fixes
Copilot 3901fc4
Merge branch 'main' into copilot/edit-predictor-selection-chapter
d-morrison 42af1d4
Merge branch 'main' into copilot/edit-predictor-selection-chapter
d-morrison 5ee7750
Merge remote-tracking branch 'origin/main' into copilot/edit-predicto…
Copilot 77ca8cb
chore: plan unresolved comment follow-ups
Copilot 1342e63
Address unresolved review comments in predictor-selection goals and a…
Copilot fcd8074
chore: plan merge-conflict resolution
Copilot f2c42d4
Merge origin/main to resolve PR conflicts
Copilot d910382
Restore page-number locator examples in citation-format guidance
Copilot f09ad6f
Merge origin/main and resolve latex-macros submodule conflict
Copilot b93c01d
Merge origin/main and resolve latex-macros submodule conflict again
Copilot 8074a34
Merge remote-tracking branch 'origin/main' into copilot/edit-predicto…
Copilot 5103e7a
Merge remote-tracking branch 'origin/main' into copilot/edit-predicto…
Copilot bd9efaa
Merge remote-tracking branch 'origin/main' into copilot/edit-predicto…
Copilot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -38,6 +38,7 @@ Imports: | |
| Suggests: | ||
| tidyverse, | ||
| conflicted, | ||
| dagitty, | ||
| dplyr, | ||
| eha, | ||
| ggfortify, | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.