search

mmcdermott / MEDS-DEV

The MEDS Decentralized Extensible Validation (MEDS-DEV) Benchmark: Establishing Reproducibility and Comparability in ML for Health
MIT License
11 stars 7 forks source link

Task `mortality/in_icu/first_24h` includes `hospital_discharge` #28

Closed rvandewater closed 2 weeks ago

rvandewater commented 1 month ago

Question for @mmcdermott and others. @prockenschaub and I were wondering why the hospital discharge is in the task definition as there should always be an icu_discharge before. This issue made the the task incompatible with the predicates of AUMCdb as it does not define hospital_discharge. I will open an issue in ACES as well.

https://github.com/mmcdermott/MEDS-DEV/blob/c235007d8394b3cc0cb13aa23014fea63e3524a9/src/MEDS_DEV/tasks/criteria/mortality/in_icu/first_24h.yaml#L25

mmcdermott commented 1 month ago

I think it would be reasonable to remove the hospital discharge requirement in favor of only requiring icu discharge. The reason for including both is that if a dataset is a bit malformed and doesn't record a separate ICU discharge if the patient is discharged straight to home from the hospital, but I think in retrospect we should include this by augmenting the code definition of icu_discharge to include both ICU and hospital discharges where present.

rvandewater commented 1 month ago

Thanks for the input! This can be changed in the future depending on ACES's leniency towards missing codes. Addressed in #27