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

central and lateral incisor tooth #2757

Merged
merged 9 commits into from
Jul 25, 2024
Merged

central and lateral incisor tooth #2757

merged 9 commits into from
Jul 25, 2024

Conversation

meghalithic
Copy link
Collaborator

@meghalithic meghalithic commented Dec 21, 2022

changing definition and EQ
issue #2719

see discussion here on 28/11/2022.

@anitacaron anitacaron linked an issue Jan 12, 2023 that may be closed by this pull request
@anitacaron anitacaron removed the request for review from shawntanzk January 30, 2023 12:31
@meghalithic meghalithic requested a review from a user February 10, 2023 04:27
@@ -168161,6 +168162,7 @@ id: UBERON:0016455
name: upper lateral secondary incisor tooth
subset: pheno_slim
synonym: "maxillary lateral secondary incisor tooth" EXACT [FMA:55724]
synonym: "secondary upper second incisor tooth" EXACT []
Copy link

Choose a reason for hiding this comment

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

Is there a dbxref to support calling this the "second" incisor tooth?

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Hi @megbalk, I don't see any edits that would "break" the ontology in this PR, but from a content perspective I recommend seeking a review from someone qualified in this subject matter.

Some of the terms do not have text definitions and the added synonyms do not have dbxrefs to check.

Regarding, "... phylogenetically number #" in the text definitions... after a cursory search online, I could not find an explanation of a phylogenetic numbering system used in this context. This may be due to a lack of domain knowledge, which is why I recommend seeking a review from someone with subject matter expertise.

The annotation property 'tooth number' mentions an ancestral count and a count along the A-P axis. I don't know if the most anterior tooth in a set is always ancestrally the first to appear. Hopefully someone with domain knowledge could help.

See also #557.

In summary, I recommend:

  • adding text definitions where they are missing
  • adding dbxrefs to support the synonyms
  • seeking someone with subject matter expertise in dentition to review the content

If there is a reference that you can share that describes phylogenetically numbering from an anatomical perspective, I'm happy to take a look and do another review.

@meghalithic
Copy link
Collaborator Author

@cmungall @RDruzinsky could I get some help here?

@github-actions
Copy link

This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR creator will be allowed to proceed with merging without explicit approval, should they wish to do so.

@meghalithic meghalithic requested review from cmungall and a user and removed request for a user March 21, 2023 08:28
@github-actions
Copy link

This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR creator will be allowed to proceed with merging without explicit approval, should they wish to do so.

@github-actions
Copy link

This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR creator will be allowed to proceed with merging without explicit approval, should they wish to do so.

@anitacaron
Copy link
Collaborator

#gogoeditdiff

@anitacaron
Copy link
Collaborator

#gogoeditdiff

Copy link

github-actions bot commented Jan 2, 2024

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/2024-01-02/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/2024-01-02/uberon-simple.owl
  • Loaded from: file:/work/src/ontology/uberon-simple.owl

Ontology imports

Ontology annotations

incisor tooth 1 http://purl.obolibrary.org/obo/UBERON_0018551

Removed

Added

incisor tooth 2 http://purl.obolibrary.org/obo/UBERON_0018552

Removed

Added

lower central incisor tooth http://purl.obolibrary.org/obo/UBERON_0018601

Removed

Added

lower central secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018568

Added

lower lateral incisor tooth http://purl.obolibrary.org/obo/UBERON_0018602

Removed

Added

lower lateral secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018570

Added

primary central incisor tooth http://purl.obolibrary.org/obo/UBERON_0018553

Removed

Added

primary lateral incisor tooth http://purl.obolibrary.org/obo/UBERON_0018554

Removed

Added

upper central incisor tooth http://purl.obolibrary.org/obo/UBERON_0018603

Removed

Added

upper central primary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018593

Added

upper central secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0016454

Added

upper lateral incisor tooth http://purl.obolibrary.org/obo/UBERON_0018604

Removed

Added

upper lateral primary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018594

Added

upper lateral secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0016455

Added

Copy link

github-actions bot commented Jan 2, 2024

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

incisor tooth 1 http://purl.obolibrary.org/obo/UBERON_0018551

Removed

Added

incisor tooth 2 http://purl.obolibrary.org/obo/UBERON_0018552

Removed

Added

lower central incisor tooth http://purl.obolibrary.org/obo/UBERON_0018601

Added

lower central secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018568

Added

lower lateral incisor tooth http://purl.obolibrary.org/obo/UBERON_0018602

Added

lower lateral secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018570

Added

primary central incisor tooth http://purl.obolibrary.org/obo/UBERON_0018553

Added

primary lateral incisor tooth http://purl.obolibrary.org/obo/UBERON_0018554

Added

upper central incisor tooth http://purl.obolibrary.org/obo/UBERON_0018603

Added

upper central primary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018593

Added

upper central secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0016454

Added

upper lateral incisor tooth http://purl.obolibrary.org/obo/UBERON_0018604

Added

upper lateral primary incisor tooth http://purl.obolibrary.org/obo/UBERON_0018594

Added

upper lateral secondary incisor tooth http://purl.obolibrary.org/obo/UBERON_0016455

Added

Copy link

github-actions bot commented Feb 3, 2024

This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR creator will be allowed to proceed with merging without explicit approval, should they wish to do so.

@anitacaron anitacaron requested a review from wdduncan May 24, 2024 08:18
@anitacaron
Copy link
Collaborator

@wdduncan could you please review this PR?

@anitacaron
Copy link
Collaborator

@wdduncan you can check the nice diff view here

Copy link

This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR creator will be allowed to proceed with merging without explicit approval, should they wish to do so.

@wdduncan
Copy link
Collaborator

This seems fine to me.

@anitacaron anitacaron merged commit b197285 into master Jul 25, 2024
1 check passed
@anitacaron anitacaron deleted the issue-2719 branch July 25, 2024 11:16
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.

incisor teeth
3 participants