upheno icon indicating copy to clipboard operation
upheno copied to clipboard

Add new partial pattern to help queries

Open rays22 opened this issue 3 years ago • 4 comments

If applied, this commit will fix #904.

rays22 avatar Mar 02 '23 16:03 rays22

@matentzn I don't think we wanted this to be a term creation pattern, correct? I thought what you wanted was to capture the standard for adding subclass axioms to existing terms to enrich the anatomy cross-references in the ontology. I don't think this should follow the standard set up for a UPheno pattern. Anna and I discussed this with @rays22 at today's meeting

sbello avatar Mar 02 '23 19:03 sbello

I think its great to have this as a pattern - remember patterns are not only to "create new terms", but also to capture/document certain modelling decisions. I have some feedback on the PR, but it is definitely good to have these as a pattern.

matentzn avatar Mar 03 '23 08:03 matentzn

@matentzn I wasn't suggesting we not have the pattern but just questioning the need to have the term creating parts like name, definition, synonyms in the pattern. Having these here suggests that this pattern could be used to generate a new term and that is not the purpose in this case.

sbello avatar Mar 03 '23 15:03 sbello

You are 100% correct on that @sbello - sorry, misunderstood you. @rays22 we only need the subclass axiom template from the generative features.

matentzn avatar Mar 03 '23 15:03 matentzn