-
Notifications
You must be signed in to change notification settings - Fork 58
juxtaglomerular complex cell merged with kidney granular cell #3109
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
base: master
Are you sure you want to change the base?
Conversation
aleixpuigb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes in the slim needs to be done in src/templates/kidney_upper_slim.csv, not in the owl file directly.
|
#gogoeditdiff |
Here's a diff of how these changes impact the classified ontology (on -base file):Ontology comparisonLeft
Right
Ontology importsOntology annotationsOMO_0003000
|
aleixpuigb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, see inline comments.
src/ontology/cl-edit.owl
Outdated
| AnnotationAssertion(Annotation(oboInOwl:hasDbXref "GO:0072052") oboInOwl:hasExactSynonym obo:CL_0000648 "juxtaglomerulus cell") | ||
| AnnotationAssertion(oboInOwl:hasExactSynonym obo:CL_0000648 "renin secreting cell") | ||
| AnnotationAssertion(oboInOwl:hasExactSynonym obo:CL_0000648 "juxtaglomerular complex cell") | ||
| AnnotationAssertion(Annotation(obo:OMO_0003000 "") Annotation(oboInOwl:hasDbXref "doi:10.1007/s00424-012-1126-7") oboInOwl:hasRelatedSynonym obo:CL_0000648 "JG cell") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The annotation property for abbreviation is not used correctly.
| AnnotationAssertion(Annotation(obo:OMO_0003000 "") Annotation(oboInOwl:hasDbXref "doi:10.1007/s00424-012-1126-7") oboInOwl:hasRelatedSynonym obo:CL_0000648 "JG cell") | ||
| AnnotationAssertion(rdfs:label obo:CL_0000648 "kidney granular cell") | ||
| EquivalentClasses(obo:CL_0000648 ObjectIntersectionOf(obo:CL_0000151 ObjectSomeValuesFrom(obo:BFO_0000050 obo:UBERON_0002113) ObjectSomeValuesFrom(obo:RO_0002215 obo:GO_0002001))) | ||
| EquivalentClasses(obo:CL_0000648 ObjectIntersectionOf(obo:CL_0002681 ObjectSomeValuesFrom(obo:BFO_0000050 obo:UBERON_0002303) ObjectSomeValuesFrom(obo:RO_0002215 obo:GO_0002001))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with the change since it is a cortical cell, and 'secreting cell' is reasoned. In the textual definition, people who know kidney anatomy will know that the glomerulus is part of the cortex. However, there might be people that don't know and might want to have it included in the textual definition (maybe in bracket after glomerulus). What do you think?
|
|
||
| AnnotationAssertion(Annotation(oboInOwl:hasDbXref "GOC:cvs") Annotation(oboInOwl:hasDbXref "GOC:tfm") Annotation(oboInOwl:hasDbXref "PMID:11457727") obo:IAO_0000115 obo:CL_0000648 "A smooth muscle cell that synthesizes, stores, and secretes the enzyme renin. This cell type are located in the wall of the afferent arteriole at the entrance to the glomerulus. While having a different origin than other kidney smooth muscle cells, this cell type expresses smooth muscle actin upon maturation.") | ||
| AnnotationAssertion(Annotation(oboInOwl:hasDbXref "GOC:cvs") Annotation(oboInOwl:hasDbXref "GOC:tfm") Annotation(oboInOwl:hasDbXref "PMID:11457727") obo:IAO_0000115 obo:CL_0000648 "A smooth muscle cell that synthesizes, stores, and secretes the enzyme renin. This cell type are located in the wall of the afferent arteriole at the entrance to the glomerulus (a structure within the renal cortex). While having a different origin than other kidney smooth muscle cells, this cell type expresses smooth muscle actin upon maturation.") | ||
| AnnotationAssertion(Annotation(oboInOwl:hasDbXref "doi:10.1007/s00424-012-1126-7") obo:OMO_0003000 obo:CL_0000648 "JG cell") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, add this term (JG cell) as related synonym, and add the AP has_synonym_type 'abbreviation' (entity IRI).
|
I would recommend to close this PR and start again, it has too many unrelated changes. |
|
This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR editor will be allowed to proceed with merging without explicit approval, should they wish to do so. |
Merge juxtaglomerular complex cell into kidney granular cell
Equivalent to will be combined in the following way: 'kidney cortical cell' and ('part of' some 'juxtaglomerular apparatus') and ('capable of' some 'renin secretion into blood stream') and keep also SubClassOf smooth muscle cell. All other logical axioms should be inferred.
JG exact synonym should be changed to related with annotation property abbreviation and with a reference.
New exact synonym: juxtaglomerular complex cell
Remove juxtaglomerular complex cell from the kidney_slim, checking if it affects any other term