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

Index Event Breakdown is not reported for subset cohorts #1071

Open
gowthamrao opened this issue Aug 29, 2023 · 3 comments
Open

Index Event Breakdown is not reported for subset cohorts #1071

gowthamrao opened this issue Aug 29, 2023 · 3 comments

Comments

@gowthamrao
Copy link
Member

To reproduce: build a cohort definition object by using CohortGenerator subset operator, run Cohort Diagnostics with index event breakdown diagnostics turned on. See output - no index event breakdown is reported for subset cohorts.

Expected: I would expect that index event breakdown is reported for subset cohort.

Justification: We expect to see changes in index event breakdown by calendar period, e.g. if a new code was introduced in ICD10 that did not exist in ICD9 - it may change the index event entry breakdown. Also changes may be observed because of some ecological reasons such as introduction of new class of drugs. By subsetting - we can empirically evaluate this.

Technical reason: in current implementation cohort definitions that do not have concept set expressions in them skip concept set diagnostics and index event breakdown. This makes sense because subset cohorts dont have concept set expression in it. But, subset are based on a primary parent cohort definition that has a concept set. I think we should allow for these diagnostics for subset cohorts (atleast for index event breakdown) by using the parent cohorts concept set.

Temporary workaround: use cohort characterization with period = day 0. This can approximate index event breakdown.

@azimov
Copy link
Collaborator

azimov commented Aug 30, 2023

Are you running with both the base cohort and the subset in the cohort definition set? The concept sets from parents should be copied to children but this obviously isn't possible if the parent cohort isn't passed to cohort diagnostics.

@gowthamrao
Copy link
Member Author

Yes - running them together in one cohortDefinitionSet

@gowthamrao
Copy link
Member Author

@azimov have you confirmed this ?

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

No branches or pull requests

2 participants