Skip to content

Comments

Initial Cohort Statistics #27

Open
larsanczoty wants to merge 18 commits intomainfrom
add_init_cohort_function
Open

Initial Cohort Statistics #27
larsanczoty wants to merge 18 commits intomainfrom
add_init_cohort_function

Conversation

@larsanczoty
Copy link
Contributor

@larsanczoty larsanczoty commented Feb 19, 2026

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
* This might make it a bit easier to read?

* Addressing PR comment
Copy link
Member

@ddbaptiste ddbaptiste left a comment

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
Copy link
Contributor

@MJC598 MJC598 left a comment

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

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