Skip to content
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

Update age timing description to note that it is always diagnosis at diagnosis #336

Merged

Conversation

allyhawkins
Copy link
Member

Closes #332

This updates the existing age_timing description added in #331 to clarify that this value will always be diagnosis if the sample disease timing is diagnosis. Is the way I described it clear?

@allyhawkins allyhawkins linked an issue Jul 25, 2024 that may be closed by this pull request
@allyhawkins allyhawkins requested a review from sjspielman July 25, 2024 20:31
Copy link
Member

@sjspielman sjspielman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I think this is clear!

@allyhawkins allyhawkins merged commit 391982b into development Jul 25, 2024
2 checks passed
@allyhawkins allyhawkins deleted the allyhawkins/diagnosis-age-timing-clarification branch July 25, 2024 21:14
allyhawkins added a commit that referenced this pull request Aug 5, 2024
…timing-clarification

Update age timing description to note that it is always diagnosis at diagnosis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update age_timing to note when diagnosis is used
2 participants