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

Add teat and teat skin #2650

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add teat and teat skin #2650

wants to merge 2 commits into from

Conversation

shawntanzk
Copy link
Collaborator

Fixes #2353

Questions/notes:

  • are nipple and teat in historical or serial homology? (I assume historical but wanted to check) -> have not added this relationship yet
  • currently there isnt any taxon restrictions, would like to have one, but have left it as a editors note for now

Tagging @ddooley & @stheil15

Fixes #2353

Would like the following:
- are nipple and teat in historical or serial homology? (I assume historical but wanted to check)
- currently there isnt any taxon restrictions, would like to have some, but have left it as a editors note for now
@shawntanzk shawntanzk requested review from RDruzinsky and a user September 13, 2022 07:12
@shawntanzk shawntanzk self-assigned this Sep 13, 2022
@github-actions
Copy link

github-actions bot commented Sep 13, 2022

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

Ontology comparison

Left

  • Ontology IRI: http://purl.obolibrary.org/obo/uberon/uberon-simple.owl
  • Version IRI: http://purl.obolibrary.org/obo/uberon/releases/2022-09-13/uberon-simple.owl
  • Loaded from: file:/work/src/ontology/../../master/src/ontology/uberon-simple.owl

Right

  • Ontology IRI: http://purl.obolibrary.org/obo/uberon/uberon-simple.owl
  • Version IRI: http://purl.obolibrary.org/obo/uberon/releases/2022-09-13/uberon-simple.owl
  • Loaded from: file:/work/src/ontology/uberon-simple.owl

Ontology imports

Ontology annotations

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

Added

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

Added

@github-actions
Copy link

github-actions bot commented Sep 13, 2022

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:/work/src/ontology/../../master/src/ontology/uberon-edit.obo

Right

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

Ontology imports

Ontology annotations

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

Added

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

Added

Copy link
Member

@cmungall cmungall left a comment

Choose a reason for hiding this comment

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

Sorry, this makes the ontology far less useful. We are splitting a single coherent concept into unconnected parts of the ontology. This will actively hinder inter operation. What if MP uses one and HP the other? How will we unify in upheno?

There are many classes in uberon where human has a different morphology from other mammals. If the differences are substantial enough, we make two classes with a common parent. But even here we must think carefully. The main ten e cost of latticization is really high. Does the use case justify it? Or is this really just a matter of giving people their own terminology?

Sorry to be negative your work in this is much appreciated.. apologies for lack of documentation on patterns.

name: teat
def: "The projection from the mammary glands of mammals from which milk flows or is ejected for the purpose of feeding young. As compared to a nipple, where the individual lactiferous ducts open directly to the outside world, a teat has lactiferous ducts that converge on a large lactiferous sac at the base of the teat and there is a single, large duct that runs through the teat with an outlet at the end of the teat." [https://orcid.org/0000-0002-1572-1316, WikipediaVersioned:Teat&oldid=1107850923]
comment: While many lay sources treat nipple and teat as synonyms, mammalogists do not. {xref="https://orcid.org/0000-0002-1572-1316"}
is_a: UBERON:0000475 ! organism subdivision
Copy link
Member

Choose a reason for hiding this comment

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

Is that all we can say? No part of?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh defs not, I 100% want a link to nipple and from what I understand from the discussion, teat is a homologous to nipple - hence the above asking if its historical or serial. As for part_of -> I'm following nipple which does not have any

@shawntanzk
Copy link
Collaborator Author

Does the use case justify it? Or is this really just a matter of giving people their own terminology?

@stheil15 & @ddooley - could you help with this please

@shawntanzk shawntanzk marked this pull request as draft September 13, 2022 14:11
@ddooley
Copy link

ddooley commented Sep 13, 2022

Over to you @stheil15 !

id: UBERON:8440079
name: teat skin
def: "The skin that is part of the teat." [https://orcid.org/0000-0001-7258-9596]
intersection_of: UBERON:0001003 ! skin epidermis
Copy link
Member

Choose a reason for hiding this comment

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

general red flag here: label, text def, and logical def should be inconcordance

in general in uberon, skin = epidermis + dermis, so we shouldn't conflate the two - even if there are some zones where dermis is lacking.

Also I don't think we can address this issue coherently until we resolve #1873. If teat/nipple is a projection of the skin, as proposed there, then teat skin = teat!

We really need clear boundaries defined for all terms

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.

add term request "teat skin"
3 participants