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

Add ecg module information #212

Merged
merged 2 commits into from
Jan 2, 2024
Merged

Add ecg module information #212

merged 2 commits into from
Jan 2, 2024

Conversation

briangow
Copy link
Contributor

@briangow briangow commented Oct 2, 2023

This PR adds an overview of the MIMIC-IV-ECG module to the website and provides details for each column in the lookup tables.

@briangow briangow requested a review from tompollard October 2, 2023 20:17
---

{{% pageinfo %}}
Please note that the free-text cardiologist notes for the ECG module will be publicly released in the MIMIC-IV-Note module at a later date. An initial set of these notes are available to our consortium members only, at this time.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not "our consortium" will mean much to people?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tompollard I agree, I've removed the mention of our consortium.

Additional details for each lookup table are provided in the pages below.

{{% pageinfo %}}
Please note that the free-text cardiologist notes for the ECG module will be publicly released in the MIMIC-IV-Note module at a later date. An initial set of these notes are available to our consortium members only, at this time.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure that "our consortium members" means anything to people.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also removed from here.

Copy link
Member

@tompollard tompollard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@briangow briangow merged commit 79e2ac2 into main Jan 2, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants