To avoid the case that we include non-redundant subclass of links in Mondo, we mondo-edit needs to be the source for subclass sync, not mondo.owl. Here the source is swapped.
Swaps the Mondo source for the subclass sync to mondo-edit.obo.
Pre-merge checklist
Documentation
Was the documentation added/updated under docs/?
[ ] Yes
[ ] No, updates to the docs were not necessary after careful consideration
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?
[ ] Yes
[ ] No, there are no functional (code-related) changes to the pipeline in the PR, so no re-run is necessary
joeflack4
commented
2 weeks ago
IGNORE PR UNTIL https://github.com/monarch-initiative/mondo/pull/7801#issue-2353417575 has been discussed
To avoid the case that we include non-redundant subclass of links in Mondo, we mondo-edit needs to be the source for subclass sync, not mondo.owl. Here the source is swapped.
For more context see https://github.com/monarch-initiative/mondo/pull/7801#issue-2353417575.
Overview
This PR:
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-ingeston this branch (afterdocker pull obolibrary/odkfull:dev), and no errors occurred?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?