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

Think about the aggregation used for the top 5 table in the report. #23

Open
LFISHER7 opened this issue Feb 9, 2023 · 0 comments
Open

Comments

@LFISHER7
Copy link
Contributor

LFISHER7 commented Feb 9, 2023

The report contains a top 5 table for each codelist, which represent the most common codes recorded for each codelist across the study period. For clinical events, these code are often distinct, but for medications, different codes can represent different doses/formulas of the same medication. For medications, it's therefore potentially more interesting to aggregate further up the hierarchy (e.g. Amoxicillin instead of Amoxicillin 250mg).

In the demo, we did this by aggregating by "vpid", but this column isn't in all codelists that will be available. In other projects, we've grouped by "bnf_code". We would need to know about the presence of these columns in the metadata from OpenCodelists if we wanted to do this aggregation.

My suggestion is that we remove any aggregation for now and treat clinical and medication events as the same. Longer term, we can get some clinical input on whether aggregation of medication events is useful and what aggregation makes most sense.

@lucyb lucyb added the OSI:V2 label Feb 9, 2023
@lucyb lucyb transferred this issue from opensafely-core/interactive.opensafely.org Feb 28, 2023
@LFISHER7 LFISHER7 removed the OSI:V2 label Jun 21, 2023
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