Add estimand clinical trials perspective to estimation section #308
Conversation
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
- Added citation for NEJM 2024 "What's in an Estimand?" article - Added new subsection on estimands in clinical trials - Included ICH E9(R1) framework and intercurrent event strategies - Cross-referenced with existing estimand literature Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Summarize NEJM article and add to estimand section
Add estimand clinical trials perspective to estimation section
Jan 5, 2026
d-morrison
requested changes
Jan 6, 2026
Owner
d-morrison
left a comment
d-morrison
left a comment
There was a problem hiding this comment.
read the article attached to the original issue #307
- Added complete author list to BibTeX entry - Corrected publication year to 2025 and added volume/pages - Rewrote clinical trials section to accurately reflect article content - Added subsections on intercurrent events and estimand strategies - Emphasized ITT as primary analysis and issues with hypothetical strategies - Added reporting recommendations from the article - Cited article throughout the updated content Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Contributor
Author
I've read the PDF article and updated the estimand section accordingly (commit 1f13072). Main changes:
|
|
d-morrison
requested changes
Jan 6, 2026
Owner
d-morrison
left a comment
d-morrison
left a comment
There was a problem hiding this comment.
split lines of text in qmd files after each phrase
Split long lines in the estimands section to have one phrase per line, following the repository's formatting guidelines for better readability and clearer diffs. Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Contributor
Author
Reformatted the text in the estimands section to have line breaks after each phrase (commit b3e4039). Each logical unit of text is now on its own line for better readability and clearer diffs. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary of Changes
Based on the actual NEJM article by Troxel et al. (2025), significantly revised the estimands section to accurately reflect the article's content:
Key updates:
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.