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

BEST4 enterocyte and subtypes #2465

Merged

Conversation

JABelfiore
Copy link
Collaborator

class hierarchy revision and 2 subclasses added

class hierarchy revision and 2 subclasses added
@JABelfiore JABelfiore linked an issue Jul 25, 2024 that may be closed by this pull request
@JABelfiore JABelfiore assigned JABelfiore and AvolaAmg and unassigned AvolaAmg Jul 25, 2024
@JABelfiore JABelfiore requested a review from AvolaAmg July 25, 2024 13:42
@dosumis
Copy link
Contributor

dosumis commented Jul 25, 2024

#gogoeditdiff

Copy link

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/cl/cl-simple.owl
  • Version IRI: http://purl.obolibrary.org/obo/cl/releases/2024-07-25/cl-simple.owl
  • Loaded from: file:/__w/cell-ontology/cell-ontology/src/ontology/cl-simple-master.owl/cl-simple.owl

Right

  • Ontology IRI: http://purl.obolibrary.org/obo/cl/cl-simple.owl
  • Version IRI: http://purl.obolibrary.org/obo/cl/releases/2024-07-25/cl-simple.owl
  • Loaded from: file:/__w/cell-ontology/cell-ontology/src/ontology/cl-simple-pr.owl/cl-simple.owl

Ontology imports

Ontology annotations

BEST4+ colonocyte http://purl.obolibrary.org/obo/CL_4047002

Added

BEST4+ enterocyte http://purl.obolibrary.org/obo/CL_4030026

Removed

Added

small intestine BEST4+ enterocyte http://purl.obolibrary.org/obo/CL_4047001

Added

Copy link

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

Ontology comparison

Left

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

Right

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

Ontology imports

Ontology annotations

BEST4+ colonocyte http://purl.obolibrary.org/obo/CL_4047002

Added

BEST4+ enterocyte http://purl.obolibrary.org/obo/CL_4030026

Removed

Added

small intestine BEST4+ enterocyte http://purl.obolibrary.org/obo/CL_4047001

Added

@dosumis
Copy link
Contributor

dosumis commented Jul 25, 2024

Looks good, but better to make part relationships to epithelium of colon/small intestine.

Also, there is some potential material on function in source papers that could go in extended description.

@anitacaron
Copy link
Contributor

Please always provide a descriptive PR title to clarify the changes in the release notes. 😄

@AvolaAmg AvolaAmg requested a review from anitacaron July 26, 2024 13:06
Copy link
Collaborator

@AvolaAmg AvolaAmg left a comment

Choose a reason for hiding this comment

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

Please see in line comments

src/ontology/cl-edit.owl Outdated Show resolved Hide resolved
src/ontology/cl-edit.owl Outdated Show resolved Hide resolved

AnnotationAssertion(Annotation(oboInOwl:hasDbXref <https://orcid.org/0000-0002-7431-4139>) Annotation(oboInOwl:hasDbXref "PMID:24223998") Annotation(oboInOwl:hasDbXref "PMID:35176508") obo:IAO_0000115 obo:CL_4030026 "An absorptive epithelial cell of the human intestine expressing bestrophin-4 (BEST4) calcium-activated ion channels.")
AnnotationAssertion(Annotation(oboInOwl:hasDbXref <https://orcid.org/0000-0002-7431-4139>) Annotation(oboInOwl:hasDbXref "PMID:24223998") Annotation(oboInOwl:hasDbXref "PMID:35176508") obo:IAO_0000115 obo:CL_4030026 "An enterocyte of the human intestine expressing bestrophin-4 (BEST4) calcium-activated ion channels.")
Copy link
Collaborator

Choose a reason for hiding this comment

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

consider to add the broad synonym
BEST4+ epithelial cell https://www.nature.com/articles/s41586-021-03852-1
BEST4+ cell
and
The related synonym
BEST4-expressing enterocyte (same paper)

Copy link
Contributor

Choose a reason for hiding this comment

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

we decided to add as exact

Copy link
Collaborator

Choose a reason for hiding this comment

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

@JABelfiore Have these terms been added as exact synonyms? I can only see BEST4+ intestinal epithelial cell, human as exact synonym

logical definition updated best4
@dosumis dosumis changed the title class hierarchy revision BEST4 enterocyte and subtypes Aug 2, 2024
src/ontology/cl-edit.owl Outdated Show resolved Hide resolved
@Caroline-99
Copy link
Collaborator

The QC is still failing. This seems to be because each of these classes (CL:4047001 and CL:4047002) is associated with multiple, conflicting definitions. I think @JABelfiore you might need to change the CL Ids manually.
@aleixpuigb could you please check that?
image

@aleixpuigb
Copy link
Collaborator

The QC is still failing. This seems to be because each of these classes (CL:4047001 and CL:4047002) is associated with multiple, conflicting definitions. I think @JABelfiore you might need to change the CL Ids manually. @aleixpuigb could you please check that? image

That's correct, Those ID were used for cycling cells.

CL IDs manually edited to prevent equivalent classes
@Caroline-99
Copy link
Collaborator

#gogoeditdiff

@Caroline-99
Copy link
Collaborator

The QC is still failing. This seems to be because each of these classes (CL:4047001 and CL:4047002) is associated with multiple, conflicting definitions. I think @JABelfiore you might need to change the CL Ids manually. @aleixpuigb could you please check that? image

@aleixpuigb it seems that all the cells that Jasmine added have been removed. I can't find them anymore on this branch. I think this might have happened when Jasmine was trying to resolve conflicts.

Adding back BEST4 enterocyte types - recovered from earlier commits - using higher IDs from @JABelfiore ID range (+50).
fixed IDs and reserialized
- Fixed IDs
- Tweaks to textual and logical defs
- renamed enterocyte of colon to colonocyte for consistency
@dosumis
Copy link
Contributor

dosumis commented Sep 13, 2024

#gogoeditdiff

@dosumis dosumis dismissed Caroline-99’s stale review September 14, 2024 09:21

need to get this out. I think good enough

@dosumis dosumis merged commit 4adcd94 into master Sep 14, 2024
1 check passed
@dosumis dosumis deleted the 2410-class-hierarchy-best4+-intestinal-epithelial-cell-human branch September 14, 2024 09:22
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.

[Class hierarchy] BEST4+ intestinal epithelial cell, human
6 participants