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

Enforce taxon constraints #3102

Merged
merged 32 commits into from
Nov 10, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
5f9eaea
Import NCBITaxon with disjointness axioms.
gouttegd Oct 27, 2023
f27ac53
Update ncbitaxon_terms based on the -edit file.
gouttegd Oct 27, 2023
2b57baf
Implement taxon constraint check in QC pipeline.
gouttegd Oct 27, 2023
b71c786
remove wings never_in_taxon Sauropsida
anitacaron Jun 20, 2023
c865ba5
Making induction relation between suture and DM mammal-only
anitacaron Jul 21, 2023
4700c98
Unsat fix: pterygoid bone NOT part of some suspensorium.
gouttegd Sep 19, 2023
79c1edc
Unsat fix: Relax taxon constraint on 'corpus luteum'.
gouttegd Sep 21, 2023
1b33f06
Unsat fix: Remove 'pericardial cavity present_in_taxon some Ciona'.
gouttegd Sep 21, 2023
feb14d0
Unsat fix: Detach 'vertebral element' from 'vertebral column'.
gouttegd Sep 22, 2023
151f02a
Unsat fix: philtrum NOT part_of some rhinarium.
gouttegd Oct 26, 2023
c6edd6a
Unsat fix: Relax taxon constraint on 'neocortex'.
gouttegd Oct 26, 2023
33b9221
Remove mapping between UBERON:0006334 and HBA:4413.
gouttegd Oct 26, 2023
91c0fa7
Unsat fix: Remove undesired taxon specifity on 'axial skeletal system'.
gouttegd Oct 26, 2023
f25f8a2
Unsat fix: Relax taxon constraint on 'dermatocranium'.
gouttegd Oct 26, 2023
88451e1
Remove mapping between UBERON:0036215 and AEO:0000154.
gouttegd Oct 26, 2023
97ed463
Unsat fix: Relax the taxon constraint on 'epithelium'.
gouttegd Oct 27, 2023
448cc2b
Unsat fix: Relax taxon constraint on 'choroidal guanine tapetum'.
gouttegd Oct 26, 2023
92d4cbe
Unsat fix: Move misplace 'present in taxon' assertion.
gouttegd Oct 26, 2023
ed43459
Unsat fix: Remove link between 'mouth' and 'respiratory system'.
gouttegd Oct 26, 2023
872bcf9
Unsat fix: Relax taxon constraint on 'olivary pretectal nucleus'.
gouttegd Oct 26, 2023
c0a1b37
Unsat fix: Remove link between 'artrhopod tibia' and 'insect leg'.
gouttegd Oct 26, 2023
8449476
Unsat fix: Remove link between 'cranium' and 'skull'.
gouttegd Oct 26, 2023
adb2596
Unsat fix: Remove the link between 'nasal skeleton' and 'facial skele…
gouttegd Oct 26, 2023
0fc2148
Unsat fix: Remove the link between 'mouth' and 'head'.
gouttegd Oct 26, 2023
21fc13f
Import both taxslim.owl and taxslim-disjoint-over-in-taxon.owl.
gouttegd Oct 30, 2023
111cc22
Merge branch 'master' into new-taxon-constraint-check
gouttegd Oct 30, 2023
dc3aa6a
Re-generate the AEO bridge.
gouttegd Oct 30, 2023
a9b6d7e
Post a comment explaining the TC violations.
gouttegd Nov 2, 2023
6f22bc1
Merge branch 'master' into new-taxon-constraint-check
gouttegd Nov 8, 2023
49501ee
Forcefully fix externally caused TC violations.
gouttegd Nov 8, 2023
44421ae
Do not update the HRA subset component upon MIR=false.
gouttegd Nov 8, 2023
c1ca725
update hra_subset component
anitacaron Nov 9, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 27 additions & 2 deletions .github/workflows/qc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,35 @@ jobs:
- uses: actions/checkout@v2

- name: Run ontology QC checks
id: check
continue-on-error: true
env:
DEFAULT_BRANCH: master
ROBOT_JAVA_ARGS: -Xmx6G
run: cd src/ontology && mkdir -p tmp reports mirror && make ROBOT_ENV='ROBOT_JAVA_ARGS=-Xmx6G' GH_ACTION=true BRI=false MIR=false IMP=false test > TESTLOG.log

- name: Reason over taxon constraints
id: explaintc
continue-on-error: true
if: steps.check.outcome == 'failure'
run: |
if [ -s src/ontology/reports/taxon-constraint-check.txt ]; then
robot explain -i src/ontology/tmp/uberon-edit-plus-tax-equivs.owl -M unsatisfiability -u all -r ELK -e taxon-unsats.md
echo "<details>\n<summary>This PR violates some taxon constraints. Here is what the reasoner has to say:</summary>\n" > comment.md
cat taxon-unsats.md >> comment.md
echo "</details>" >> comment.md
exit 1
fi
- name: Post explanation for taxon constraint violations
if: steps.explaintc.outcome == 'failure'
uses: NejcZdovc/comment-pr@v2
with:
github_token: ${{secrets.GITHUB_TOKEN}}
file: "../../comment.md"
identifier: "TAXON_CONSTRAINTS_REASONING"

- name: Last rows of QC report
if: ${{ failure() }}
run: tail -20 src/ontology/TESTLOG.log
if: steps.check.outcome == 'failure'
run: |
tail -20 src/ontology/TESTLOG.log
exit 1
10 changes: 9 additions & 1 deletion src/ontology/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ all_main: $(MAIN_FILES)
# ----------------------------------------


IMPORTS = pr cl go envo ro bspo chebi pato bfo ncbitaxon nbo orcidio omo
IMPORTS = pr cl go envo ro bspo chebi pato bfo ncbitaxon ncbitaxondisjoints nbo orcidio omo

IMPORT_ROOTS = $(IMPORTDIR)/merged_import
IMPORT_OWL_FILES = $(foreach n,$(IMPORT_ROOTS), $(n).owl)
Expand Down Expand Up @@ -496,6 +496,14 @@ mirror-ncbitaxon: | $(TMPDIR)
mv [email protected] $(TMPDIR)/[email protected]; fi


## ONTOLOGY: ncbitaxondisjoints
.PHONY: mirror-ncbitaxondisjoints
.PRECIOUS: $(MIRRORDIR)/ncbitaxondisjoints.owl
mirror-ncbitaxondisjoints: | $(TMPDIR)
if [ $(MIR) = true ] && [ $(IMP) = true ]; then $(ROBOT) convert -I http://purl.obolibrary.org/obo/ncbitaxon/subsets/taxslim-disjoint-over-in-taxon.owl -o [email protected] &&\
mv [email protected] $(TMPDIR)/[email protected]; fi


## ONTOLOGY: nbo
.PHONY: mirror-nbo
.PRECIOUS: $(MIRRORDIR)/nbo.owl
Expand Down
7 changes: 1 addition & 6 deletions src/ontology/bridge/uberon-bridge-to-aeo.obo
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
ontology: uberon/bridge/uberon-bridge-to-aeo
data-version: 2023-10-27
data-version: 2023-10-30
property_value: seeAlso "http://purl.obolibrary.org/obo/uberon/references/reference_0000026" xsd:anyURI

[Term]
Expand Down Expand Up @@ -697,11 +697,6 @@ id: AEO:0001010 !
property_value: IAO:0000589 "differentiating neuroepithelium (AEO)" xsd:string
equivalent_to: UBERON:0034707 ! differentiating neuroepithelium

[Term]
id: AEO:0000154 !
property_value: IAO:0000589 "anatomical surface region (AEO)" xsd:string
equivalent_to: UBERON:0036215 ! anatomical surface region

[Typedef]
id: part_of
xref: BFO:0000050
Expand Down
10 changes: 1 addition & 9 deletions src/ontology/bridge/uberon-bridge-to-aeo.owl
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
xmlns:terms="http://purl.org/dc/terms/"
xmlns:oboInOwl="http://www.geneontology.org/formats/oboInOwl#">
<owl:Ontology rdf:about="http://purl.obolibrary.org/obo/uberon/bridge/uberon-bridge-to-aeo.owl">
<owl:versionIRI rdf:resource="http://purl.obolibrary.org/obo/uberon/bridge/uberon-bridge-to-aeo/2023-10-27/uberon/bridge/uberon-bridge-to-aeo.owl"/>
<owl:versionIRI rdf:resource="http://purl.obolibrary.org/obo/uberon/bridge/uberon-bridge-to-aeo/2023-10-30/uberon/bridge/uberon-bridge-to-aeo.owl"/>
<rdfs:seeAlso rdf:datatype="http://www.w3.org/2001/XMLSchema#anyURI">http://purl.obolibrary.org/obo/uberon/references/reference_0000026</rdfs:seeAlso>
</owl:Ontology>

Expand Down Expand Up @@ -1118,8 +1118,6 @@

<owl:Class rdf:about="http://purl.obolibrary.org/obo/AEO_0000154">
<owl:equivalentClass rdf:resource="http://purl.obolibrary.org/obo/UBERON_0002416"/>
<owl:equivalentClass rdf:resource="http://purl.obolibrary.org/obo/UBERON_0036215"/>
<obo:IAO_0000589 rdf:datatype="http://www.w3.org/2001/XMLSchema#string">anatomical surface region (AEO)</obo:IAO_0000589>
<obo:IAO_0000589 rdf:datatype="http://www.w3.org/2001/XMLSchema#string">integumental system (AEO)</obo:IAO_0000589>
<oboInOwl:id rdf:datatype="http://www.w3.org/2001/XMLSchema#string">AEO:0000154</oboInOwl:id>
</owl:Class>
Expand Down Expand Up @@ -2531,12 +2529,6 @@
<!-- http://purl.obolibrary.org/obo/UBERON_0034707 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/UBERON_0034707"/>



<!-- http://purl.obolibrary.org/obo/UBERON_0036215 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/UBERON_0036215"/>
</rdf:RDF>


Expand Down
7 changes: 0 additions & 7 deletions src/ontology/bridge/uberon-bridge-to-dmba.obo
Original file line number Diff line number Diff line change
Expand Up @@ -1319,13 +1319,6 @@ intersection_of: BFO:0000050 NCBITaxon:10090 {name="anterior nucleus of hypothal
property_value: IAO:0000589 "posterior part of anterior hypothalamic nucleus" xsd:string
property_value: status "Verified" xsd:string {source="https://ror.org/03cpe7c52"}

[Term]
id: DMBA:16271
intersection_of: UBERON:0000435 {name="lateral tuberal nucleus"}
intersection_of: BFO:0000050 NCBITaxon:10090 {name="lateral tuberal nucleus"}
property_value: IAO:0000589 "lateral tuberal nucleus" xsd:string
property_value: status "Verified" xsd:string {source="https://ror.org/03cpe7c52"}

[Term]
id: DMBA:16308
intersection_of: UBERON:0001894 {name="diencephalon"}
Expand Down
31 changes: 0 additions & 31 deletions src/ontology/bridge/uberon-bridge-to-dmba.owl
Original file line number Diff line number Diff line change
Expand Up @@ -5723,37 +5723,6 @@



<!-- http://purl.obolibrary.org/obo/DMBA_16271 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/DMBA_16271">
<owl:equivalentClass rdf:nodeID="genid1129"/>
<obo:IAO_0000589>lateral tuberal nucleus</obo:IAO_0000589>
<oboInOwl:status>Verified</oboInOwl:status>
</owl:Class>
<owl:Class rdf:nodeID="genid1129">
<owl:intersectionOf rdf:parseType="Collection">
<rdf:Description rdf:about="http://purl.obolibrary.org/obo/UBERON_0000435"/>
<owl:Restriction>
<owl:onProperty rdf:resource="http://purl.obolibrary.org/obo/BFO_0000050"/>
<owl:someValuesFrom rdf:resource="http://purl.obolibrary.org/obo/NCBITaxon_10090"/>
</owl:Restriction>
</owl:intersectionOf>
</owl:Class>
<owl:Axiom>
<owl:annotatedSource rdf:resource="http://purl.obolibrary.org/obo/DMBA_16271"/>
<owl:annotatedProperty rdf:resource="http://www.w3.org/2002/07/owl#equivalentClass"/>
<owl:annotatedTarget rdf:nodeID="genid1129"/>
<rdfs:label>lateral tuberal nucleus</rdfs:label>
</owl:Axiom>
<owl:Axiom>
<owl:annotatedSource rdf:resource="http://purl.obolibrary.org/obo/DMBA_16271"/>
<owl:annotatedProperty rdf:resource="http://www.geneontology.org/formats/oboInOwl#status"/>
<owl:annotatedTarget>Verified</owl:annotatedTarget>
<oboInOwl:source>https://ror.org/03cpe7c52</oboInOwl:source>
</owl:Axiom>



<!-- http://purl.obolibrary.org/obo/DMBA_16308 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/DMBA_16308">
Expand Down
36 changes: 27 additions & 9 deletions src/ontology/components/hra_subset.owl
Original file line number Diff line number Diff line change
Expand Up @@ -14131,15 +14131,6 @@



<!-- http://purl.obolibrary.org/obo/UBERON_8440041 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/UBERON_8440041">
<obo:RO_0002175 rdf:resource="http://purl.obolibrary.org/obo/NCBITaxon_9606"/>
<oboInOwl:inSubset rdf:resource="http://purl.obolibrary.org/obo/uberon/core#human_reference_atlas"/>
</owl:Class>



<!-- http://purl.obolibrary.org/obo/UBERON_8450002 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/UBERON_8450002">
Expand Down Expand Up @@ -14209,6 +14200,33 @@
<obo:RO_0002175 rdf:resource="http://purl.obolibrary.org/obo/NCBITaxon_9606"/>
<oboInOwl:inSubset rdf:resource="http://purl.obolibrary.org/obo/uberon/core#human_reference_atlas"/>
</owl:Class>



<!-- http://purl.obolibrary.org/obo/UBERON_8600019 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/UBERON_8600019">
<obo:RO_0002175 rdf:resource="http://purl.obolibrary.org/obo/NCBITaxon_9606"/>
<oboInOwl:inSubset rdf:resource="http://purl.obolibrary.org/obo/uberon/core#human_reference_atlas"/>
</owl:Class>



<!-- http://purl.obolibrary.org/obo/UBERON_8600020 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/UBERON_8600020">
<obo:RO_0002175 rdf:resource="http://purl.obolibrary.org/obo/NCBITaxon_9606"/>
<oboInOwl:inSubset rdf:resource="http://purl.obolibrary.org/obo/uberon/core#human_reference_atlas"/>
</owl:Class>



<!-- http://purl.obolibrary.org/obo/UBERON_8600021 -->

<owl:Class rdf:about="http://purl.obolibrary.org/obo/UBERON_8600021">
<obo:RO_0002175 rdf:resource="http://purl.obolibrary.org/obo/NCBITaxon_9606"/>
<oboInOwl:inSubset rdf:resource="http://purl.obolibrary.org/obo/uberon/core#human_reference_atlas"/>
</owl:Class>
</rdf:RDF>


Expand Down
Loading
Loading