Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Add mimic-iv-note (v2.2) loading of postgres #1467

Merged

Conversation

DopamineLcy
Copy link
Contributor

Add mimic-iv-note (v2.2) loading method of postgres, thank you!

@DopamineLcy
Copy link
Contributor Author

BTW, if I add the constraint of "hadm_id" in "radiology" NOT NULL, an error will occur, which is not consistent with https://mimic.mit.edu/docs/iv/modules/note/radiology/, maybe the data should be checked.
So I remove the NOT NULL constraint of "hadm_id" in "radiology".
Thank you very much.

@alistairewj
Copy link
Member

This looks great, thank you! Going to merge this in and update any data types that may have changed since we described the data on the website (including the NOT NULL constraint like you noted).

@alistairewj alistairewj merged commit 4b28df1 into MIT-LCP:mimiciv_v2.2_updates Jan 15, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants