monarch-initiative / mondo-ingest

Coordinating the mondo-ingest with external sources
https://monarch-initiative.github.io/mondo-ingest/
6 stars 3 forks source link

OMIM: Annotated 'included' synonyms using a different property #616

Closed joeflack4 closed 4 months ago

joeflack4 commented 4 months ago

Related:

Overview

Update: properties.txt to list this new annotation property being used by OMIM to denote these special synonyms. This ensures that these properties are not removed during the component OWL creation process.

Pre-merge checklist

Documentation

Was the documentation added/updated under docs/?

QC

Was the full pipeline run before submitting this PR using sh run.sh make build-mondo-ingest on this branch (after docker pull obolibrary/odkfull:dev), and no errors occurred?

Build PR:

New Packages

Were any new Python packages added?

Were any other non-Python packages added?

PR Review and Conversations Resolved

Has the PR been sufficiently reviewed by at least 1 team member of the Mondo Technical team and all threads resolved?

twhetzel commented 4 months ago

I have now Approved the Data Build for this feature branch.

joeflack4 commented 4 months ago

@matentzn I didn't get your review, but in the SOP we don't have a stipulated minimum reviewer count. And it's a small PR, build passed, Trish and I reviewed, so I'm gonna merge this one.