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.
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
Initial federated COVID-rich ICU database documentation. #209
base: main
Are you sure you want to change the base?
Initial federated COVID-rich ICU database documentation. #209
Changes from 13 commits
a8fd7f8
d886750
c720f84
d3104c2
f3e35fc
1a96d34
0b8287f
f2580ef
1d5e663
a94bf9e
c397bb6
4a74db5
9739e7e
1a81904
e6fa4d1
9cc1e41
d54af00
a34c0e9
edb72f0
2f8806e
5f05e6f
908c7c3
0f0db10
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all of the data for this subset should be ICD-10 - can't imagine NW using ICD-9 in 2020 onward - perhaps make it clear the description of ICD-9 is just for informational purposes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, added a clarifying statement to indicate ICD 9 codes are just for informational purposes, after mandate of ICD10 in 2015 based on CMS.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would be useful to add
itemid
and the LOINC for these examplesThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm I don't see it? something wrong with my viewer or did the commit not go through?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is actually part of commit 2f8806e.