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

Improve UMLS coverage #50

Merged
merged 57 commits into from
Oct 10, 2022
Merged

Improve UMLS coverage #50

merged 57 commits into from
Oct 10, 2022

Conversation

gaurav
Copy link
Collaborator

@gaurav gaurav commented Jul 18, 2022

This PR will implement the plan in TranslatorSRI/NodeNormalization#119 (comment) to (1) create a compendium of UMLS IDs, labels and types for UMLS IDs that weren't included in other compendia, and (2) produce a report that will help us find sections of the UMLS that should be incorporated into Babel in other compendia.

It also fixes a minor bug in which "UMLS" was not listed as a prerequisite for "chemical", which caused "chemical" in some cases to run before the UMLS ids, labels and synonyms had been produced.

@gaurav gaurav force-pushed the improve-umls-coverage branch from 09aa9e2 to 2028daa Compare August 29, 2022 02:37
@gaurav gaurav changed the base branch from remove-efo to write-out-provenance-in-concords August 29, 2022 02:38
@gaurav gaurav force-pushed the write-out-provenance-in-concords branch from 048012d to 5b63ffa Compare August 29, 2022 03:55
@gaurav gaurav force-pushed the improve-umls-coverage branch from fa7fb28 to 2e9160b Compare August 29, 2022 04:02
@gaurav gaurav changed the base branch from master to add-umls-to-taxon September 29, 2022 20:03
@gaurav gaurav force-pushed the improve-umls-coverage branch from 2b42e60 to 5d2cdb8 Compare September 29, 2022 20:06
Base automatically changed from add-umls-to-taxon to master October 3, 2022 03:03
@gaurav gaurav marked this pull request as ready for review October 4, 2022 03:59
@gaurav gaurav requested a review from cbizon October 4, 2022 03:59
@gaurav gaurav requested a review from cbizon October 10, 2022 16:36
@gaurav gaurav merged commit a55ce2f into master Oct 10, 2022
@gaurav gaurav deleted the improve-umls-coverage branch October 10, 2022 21:38
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.

3 participants