-
Notifications
You must be signed in to change notification settings - Fork 47
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
vocab load fails on null values in vocabulary.vocabulary_reference about Health Plans #56
Comments
Thanks for that @turbomam Helped move me forward. Seems like the MIMIC dataset and these sets of OMOP tools have fallen out of date. I find the pgtap tests to fail as well since they're expecting certain counts, but not getting them. |
This was built using the v5 model (early 2018) so it may be related to more recent changes in the vocab. Unfortunately work on this project was paused for a time due to competing priorities, but we plan to revisit it over the coming year. |
We converted MIMIC in OMOP v5.3. The new OMOP version is OMOP v6.0 but all the OHDSI tools are still not working in OMOP 6.0. So we will wait a little. |
Thanks all for the feedback. I have also raised some issues for the Synthea ETL, like OHDSI/ETL-Synthea#19. Are any of you in a position to reply to any of them? I'm doing these conversions for a paper that will be submitted soon. I'm using 5.x schemas. Thanks for the great work. |
It will likely be the case that the OHDSI tools will not support the CDM 6.0 until early 2020. That being said we have plans to release an update to ETL-Synthea that converts to 6.0 much sooner. It will then be used by OHDSI development teams for regression testing of tools as CDM 6.0 is adopted. |
Thanks, this is a great tool.
When running the vocab load part of your pipeline, I got
Which I think I traced down to
So I did
And everything seems OK now
I'm using every vocabulary that doesn't require a license. I did the cpt pull from UMLS.
The text was updated successfully, but these errors were encountered: