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

common_stage: arterial measurement labels #19

Closed
michela-meister opened this issue Jul 15, 2022 · 1 comment
Closed

common_stage: arterial measurement labels #19

michela-meister opened this issue Jul 15, 2022 · 1 comment

Comments

@michela-meister
Copy link

Hello,
We are trying to work with data on oxygen saturation in arterial blood (concept ID 20000800 the hirid variable reference spreadsheet). In the appendix of the original HiRID paper, the authors explain that the raw data includes venous measurements that were incorrectly labeled as arterial, and they had to correct these measurements during preprocessing (which they do here with with the function change_arterial_to_venous). Do the common_stage matrices include the correct labels for arterial measurements?

@hugoych
Copy link
Collaborator

hugoych commented Jul 19, 2022

Hi Michela,
Thanks for using our benchmark!

The common stage matrices are not corrected for possible arterial/venous artifacts beyond out-of-range removal from the varref.tsv table as other variables. This is because the pipeline up to the common stage aims to not depend on any design/modeling choices as discussed in the paper. Indeed the change_arterial_to_venous function, from the original repo, relies on a given threshold parameter to define mislabeled arterial S02 measurements. Because this only affects less than 400 measurements out of more than 3M, we preferred not to include this step. For the same reason, we only removed out-of-range height and weight and did not include the correct_weight_height function from the original repo.
I hope this answers your question!

@hugoych hugoych closed this as completed Jul 26, 2022
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