forked from eth-mds/ricu
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add variables to PICDS #8
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Done files:
To-Do
|
10 tasks
manuelburger
added a commit
that referenced
this pull request
Mar 22, 2024
This reverts commit 7824352.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
cai
The config description is "calcium iodized". The corresponding row in the picdb
d_labitems
iscai
The config description is "calcium". The corresponding row in the picdb
d_labitems
isNote the english description is equal to that of
ca
. The entries in picdblabevents
are mmol/L, so we need to transform.dbp and sbp
Note, there are also
in the surgery tables.
etco2
Only exists for surgery tables
tco2
Note that the unit of tco2(P) in picdb is
mmol/L
, different tomEq/L
. But this is probably the same, possibly as the charge is +-1? @martinAlso, means appear to agree between mimic / picdb. mimic is
meq/l
.pco2
What is pco2(T) vs pco2?
map
This does not appear in picdb, but can be computed as 2/3 * DBP + 1/3 SBP. Is this correct @martin? How to implement this in ricu?
hr
already there. Also
We also add
temp
There is an entry in
labevents
. Why? Values appear to make sense though:We leave as is.
fio2
Note that this appears to be used in
value make sense:
There's also FO2Hb, with mean value
hbco
appears to make sense - leave as is
lact
appears to make sense - leave as is
methb
appears to make sense
ph
removed everything except
What is "(T)" here? @martin
po2
5210 also contains strings "alkaline" etc., however the unit of 5208 is
%
. Also, the median is ~30.sex
gender column in
patients
table.ethnic
Everyone's asian.
age
patients table has a
dob
variable, same as mimic.death
same as mimic.
death_icu
TODO
death_hosp
TODO
adm
did not find anything.
weight
Get rid of labevents entry
height
Get rid of labevents entry
alb
alp
gsehd guet us
alt
both U/L
we excluded
What is the difference @martin? This has unit
μmol/L
.ast
This is probably a typo (asparate instead of aspartate)
bicar
looks good
bili
both times
μmol/L
.bili_dir
again in mumol/L
bun
found nothing
What is
ck
ckmb
conversion factor from https://onlineconversion.vbulletin.net/forum/main-forums/convert-and-calculate/4310-u-l-to-ng-ml-conversion?p=86907#post86907.
Medians between mimic/picdb are similar after conversion.
cl
crea
added conversion.
crp
looks good
glu
added conversion
k
7023 has 2 entries without units, so we excluded it.
mg
We added the same conversion as hirid. The eicu conversion does not make sense, should be multiplication instead of division. However, the densities look reasonable. What gives? TODO: Check densities of picdb.
na
The non-whole blood values are lower @martin, so we removed them.
phos
added conversion from hirid
tnt
Need to convert from picogram to nanogram. Why are values often similar? Also, often "<0.003" in
"VALUE"
andnan
inVALUENUM
.tri
looks good
samp