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

Fix taxon constraint violations #3455

Merged
merged 4 commits into from
Dec 24, 2024
Merged

Conversation

gouttegd
Copy link
Collaborator

This PR fixes some problems introduced by the recent #3445 PR, the most important being that the new terms for crab and lobster nerves are causing taxon constraint violations as explained in #3454. We fix that by replacing:

'in taxon' some Astacidea
'in taxon' some Brachyura

with

'in taxon' some Pleocyemata

We also explicitly import the Pleocyemata term from NCBITaxon, and fix a few incorrectly formatted cross-references (such as [PMID: 17009928], with a spurious space after the colon).

closes #3454

Recently introduced terms describing some crustacean nerves have been
asserted to be simultaneously 'in_taxon some Astacidea' and 'in_taxon
some Brachyura'.

Astacidea (NCBITaxon:6712) and Brachyura (NCBITaxon:6725) are *sibling*
taxa, so such assertions are contradicting each other: a term that is
'in_taxon some Astacidae' or it is 'in_taxon some Brachyura', but it
cannot be 'in_taxon' both taxa simultaneously.

Remember that 'in_taxon some X' means that the term is *specific to the
X taxon*, exclusive of any other taxon that is not X itself or a
descendant of X.

Assuming that the terms refer to nerves that are in fact present both in
Astacidea and Brachyura, we replace for all affected terms the faulty
'in_taxon' assertions by a single assertion 'in_taxon some Pleocyemata',
where Pleocyemata (NCBITaxon:6692) is the parent taxon of both Astacidea
and Brachyura.

We also explicitly import NCBITaxon:6692, since it is used in those
assertions.
The -edit file contained some weirdly formatted line with trailing
spaces, some tags improperly ordered, and some incorrectly formatted
xrefs (possibly the result of some manual editing with a text editor?),
so we re-serialize it.
@gouttegd gouttegd self-assigned this Dec 23, 2024
@aleixpuigb
Copy link
Collaborator

#gogoeditdiff

Copy link

Here's a diff of how these changes impact the classified ontology (on -base file):

Ontology comparison

Left

  • Ontology IRI: http://purl.obolibrary.org/obo/uberon/uberon-base.owl
  • Version IRI: http://purl.obolibrary.org/obo/uberon/releases/2024-12-23/uberon-base.owl
  • Loaded from: file:/__w/uberon/uberon/src/ontology/uberon-base-main.owl/uberon-base.owl

Right

  • Ontology IRI: http://purl.obolibrary.org/obo/uberon/uberon-base.owl
  • Version IRI: http://purl.obolibrary.org/obo/uberon/releases/2024-12-23/uberon-base.owl
  • Loaded from: file:/__w/uberon/uberon/src/ontology/uberon-base-pr.owl/uberon-base.owl

Ontology imports

Ontology annotations

NCBITaxon_6692 http://purl.obolibrary.org/obo/NCBITaxon_6692

Added

NCBITaxon_6712 http://purl.obolibrary.org/obo/NCBITaxon_6712

Removed

NCBITaxon_6752 http://purl.obolibrary.org/obo/NCBITaxon_6752

Removed

commissural ganglion (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910021

Removed

Added

dorsal gastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910011

Removed

Added

esophageal ganglion (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910020

Removed

Added

esophageal nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910019

Removed

Added

gastropyloric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910012

Removed

Added

inferior esophageal nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910017

Removed

Added

junction of superior esophageal nerve and stomatogastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910022

Removed

Added

lateral gastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910013

Removed

Added

lateral ventricular nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910014

Removed

Added

median ventricular nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910023

Removed

Added

pyloric dilator nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910016

Removed

Added

pyloric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910015

Removed

Added

stomatogastric ganglion http://purl.obolibrary.org/obo/UBERON_8910001

Removed

Added

stomatogastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910010

Removed

Added

superior esophageal nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910018

Removed

Added

Copy link

Here's a diff of your edit file (unreasoned)

Ontology comparison

Left

  • Ontology IRI: http://purl.obolibrary.org/obo/uberon/core.owl
  • Version IRI: None
  • Loaded from: file:/__w/uberon/uberon/master/src/ontology/uberon-edit.obo

Right

  • Ontology IRI: http://purl.obolibrary.org/obo/uberon/core.owl
  • Version IRI: None
  • Loaded from: file:/__w/uberon/uberon/branch/src/ontology/uberon-edit.obo

Ontology imports

Ontology annotations

NCBITaxon_6712 http://purl.obolibrary.org/obo/NCBITaxon_6712

Removed

NCBITaxon_6752 http://purl.obolibrary.org/obo/NCBITaxon_6752

Removed

Pleocyemata http://purl.obolibrary.org/obo/NCBITaxon_6692

Added

commissural ganglion (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910021

Removed

Added

dorsal gastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910011

Removed

Added

esophageal ganglion (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910020

Removed

Added

esophageal nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910019

Removed

Added

gastropyloric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910012

Removed

Added

inferior esophageal nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910017

Removed

Added

junction of superior esophageal nerve and stomatogastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910022

Removed

Added

lateral gastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910013

Removed

Added

lateral ventricular nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910014

Removed

Added

median ventricular nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910023

Removed

Added

pyloric dilator nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910016

Removed

Added

pyloric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910015

Removed

Added

stomatogastric ganglion http://purl.obolibrary.org/obo/UBERON_8910001

Removed

Added

stomatogastric nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910010

Removed

Added

superior esophageal nerve (sensu Cancer borealis) http://purl.obolibrary.org/obo/UBERON_8910018

Removed

Added

rays22
rays22 previously approved these changes Dec 23, 2024
Copy link
Collaborator

@rays22 rays22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine (if in_taxon is used as intended).

Since STG is an abbreviation for "stomatogastric ganglion", it is more
appropriate to make it a RELATED synonym (rather than an EXACT synonym),
and to explicitly flag it with the "abbreviation" (OMO:0003000) synonym
type (as per the Uberon editor SOP).

Likewise for similar abbreviations on crustacean nerves.
@gouttegd gouttegd merged commit 9af7368 into master Dec 24, 2024
2 checks passed
@gouttegd gouttegd deleted the fix-taxon-constraint-violations branch December 24, 2024 11:31
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.

Newly introduced crab and lobster terms violate taxon constraints
3 participants