-
Notifications
You must be signed in to change notification settings - Fork 113
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?
Conversation
I'm not sure that "freely available" is a good description and would take those words out. In any event this database isn't yet available at all. Please add the following at the top of the page (maybe it should be added to each sub-page too?)
It is not clear to me what the long-term relationship will be between this data and "MIMIC-IV" as we know it. I think it would be unfortunate to have two sets of documentation (that both have to be maintained and that would confuse visitors). At the same time we don't want to give the impression that the new COVID stuff is already integrated in MIMIC-IV. In terms of the website structure, I'd suggest one of two things:
As this database isn't released (nor is it a replacement for MIMIC-IV), it shouldn't be given "top billing" above MIMIC-IV, and at the same time the hope is that eventually this will be part of MIMIC-IV (it's clearly not separate from MIMIC-IV in the way that MIMIC-III is.) @tompollard how would you feel about these options? Additionally, someone suggested that "Federated" might not be a good term since it has a specific technical meaning when talking about databases. Perhaps "Multi-center" could be used instead? |
thanks @bemoody for the suggestions. I have updated the changes detailed in the commits. |
@tompollard let us know if the changes look ok and we can deploy the changes on the website. |
After discussions with @rgmark, @bemoody, @tompollard and @alistairewj, I have restructured the mimic-northwestern multi-center database to the main docs page, and replaced mentions of "federated" with "multi-center" to avoid confusion with another technical term under the term "federated learning". This is a version that tries to address most of the points raised, though I think the website could be reviewed for any further suggestions either on formatting or naming etc. |
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.
some comments here and there
`icd_code` is the International Coding Definitions (ICD) code. | ||
|
||
### `icd_version` | ||
There are two versions for this coding system: version 9 (ICD-9) and version 10 (ICD-10). These can be differentiated using the `icd_version` column. [ICD-9](https://www.cms.gov/Medicare/Coding/ICD9ProviderDiagnosticCodes/codes) and [ICD-10](https://www.cms.gov/Medicare/Coding/ICD10/2019-ICD-10-CM.html) diagnosis codes are acquired from Centers for Medicare & Medicaid Services (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.
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.
|
||
We provide a list of common COVID-19 tests and measurements in the database, as defined by LOINC terminology, below: | ||
|
||
- SARS-CoV-2 (COVID-19) [Presence] in Specimen by Organism specific culture |
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 examples
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.
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.
After discussions with @rgmark regarding the |
This pulls request provides initial documentation to the tables and standard terminologies included as part of MIMIC-Fed, a large, freely available federated COVID-rich ICU database comprising deidentified health-related data from Beth Israel Deaconess Medical Center (BIDMC) and Northwestern Memorial HealthCare (NMHC).
The following tables are included as part of the hospital and icu modules in version 0.1.0 of the federated database: