obophenotype / upheno

The Unified Phenotype Ontology (uPheno) integrates multiple phenotype ontologies into a unified cross-species phenotype ontology.
https://obophenotype.github.io/upheno/
Creative Commons Zero v1.0 Universal
75 stars 17 forks source link

Review: abnormalMorphologyInLocation #317

Closed srobb1 closed 5 years ago

srobb1 commented 5 years ago

abnormalMorphologyInLocation.yaml

I am confused about the 'position' class declaration:

classes:
  position: PATO:0000140

I don't see this mentioned anywhere else in the pattern.

Sofia

matentzn commented 5 years ago

Good catch! Will fix this.

matentzn commented 5 years ago

Please leave this ticket open until the review of the above pattern is completed by 5 or more editors.

ybradford commented 5 years ago

this should probably also be limited to anatomical structures (entity is BFO:0000001)

nicolevasilevsky commented 5 years ago

pattern_name: abnormalMorphologyOfAnatomicalEntityInLocation pattern_iri: http://purl.obolibrary.org/obo/upheno/patterns-dev/abnormalMorphologyOfAnatomicalEntityInLocation.yaml description: ""

classes: abnormal: PATO:0000460 anatomical entity: UBERON:0001062 morphology: PATO:0000051

relations: inheres_in: RO:0000052 has_modifier: RO:0002573 has_part: BFO:0000051 part_of: BFO:0000050

annotationProperties: exact_synonym: oio:hasExactSynonym

vars: anatomical_entity: "'anatomical entity'" location: "'anatomical entity'"

name: text: "morphological abnormality of the %s in the %s" vars:

annotations:

def: text: "A morphological abnormality of the %s in the %s." vars:

equivalentTo: text: "'has_part' some (morphology and ('inheres_in' some (%s and ('part_of' some %s))) and ('has_modifier' some abnormal))" vars:

@obophenotype/phenotype-editors could you all review this pattern above and sign off on it in the spreadsheet. Nico hopes to have these all reviewed by mid-March, in time for the workshop at Biocuration2019.

chris-grove commented 5 years ago

Can we change the exact synonym to "Abnormal morphology of %s in the %s"? I don't think "Abnormal %s in the %s" is sufficiently specific

matentzn commented 5 years ago

Comments consolidated in http://purl.obolibrary.org/obo/upheno/patterns-dev/abnormalMorphologyOfAnatomicalEntityInLocation.yaml closing for now