Skip to content

Initial Cohort Statistics - #27

Merged
larsanczoty merged 20 commits into
mainfrom
add_init_cohort_function
Feb 24, 2026
Merged

Initial Cohort Statistics #27
larsanczoty merged 20 commits into
mainfrom
add_init_cohort_function

Conversation

@larsanczoty

@larsanczoty larsanczoty commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Creates a function to pull statistics from the initial cohort

What Wrike task is this associated with?

Checklist before merging

  • If adding a core feature, I've added related tests.
  • This is part of a product update, and I've added an explanation of what is different to the changelog.

@larsanczoty
larsanczoty requested a review from a team February 19, 2026 17:34
Comment thread Syndemics/R/init_cohort_stats.R Outdated
Comment thread Syndemics/R/init_cohort_stats.R Outdated
Comment thread Syndemics/R/init_cohort_stats.R Outdated
Comment thread Syndemics/R/init_cohort_stats.R Outdated
* This might make it a bit easier to read?

* Addressing PR comment

@ddbaptiste ddbaptiste left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Before merging this into the package, I just want to check that it functions the same? I just noticed after it was merged in that now we don't use the structure age_stats at all.

MJC598
MJC598 previously approved these changes Feb 23, 2026

@MJC598 MJC598 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving so I can refactor the package a bit for easier usage

@larsanczoty
larsanczoty requested a review from MJC598 February 24, 2026 14:53

@MJC598 MJC598 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Quick comment. Answer and I can approve.

Comment thread Syndemics/R/init_cohort_stats.R
@larsanczoty
larsanczoty merged commit 165e9bd into main Feb 24, 2026
5 checks passed
@larsanczoty
larsanczoty deleted the add_init_cohort_function branch February 24, 2026 16:09
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.

3 participants