IHTSDO / health-data-analytics

Snolytical - Health Data Analytics Demonstrator
Other
41 stars 11 forks source link

Is MinAge for time of encounter or at present? #8

Open sophie-w opened 4 years ago

sophie-w commented 4 years ago

For cohort selection - when I set MinAge, will it filter based on age at the encounter which matches the ECL, or current age (which will not be relevant for historic data)? Thanks

kaicode commented 4 years ago

Hi @sophie-w, thank you for your insightful question!

The minAge parameter in the cohort selection API is the current age of the patient. The relevant section of code in the 1.2.0 release is here.

Now I think about it I agree that a more relevant selection criterion for longitudinal data would be the age of the patient at the time of the encounter.

It could be useful to have two sets of parameters: currentMinAge & currentMax and alsoencounterMinAge & encounterMaxAge to select patients based on their age at the time of the encounter. Also the Swagger API page could really do with a short description for each parameter so users are not left guessing.

I'll mark this as an enhancement request. There is no development resources assigned to this tool at the moment but someone in the community may contribute or someone in SNOMED International may find some time.

Kind regards, Kai