-
Notifications
You must be signed in to change notification settings - Fork 39
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
membrane region design pattern results in incoherency when valid reflexivity axioms added #12926
Comments
Hmmmm. I'm fond of this pattern. I think it's pretty useful. Most biologists are happy to call membrane regions membranes (e.g. presynaptic membrane). Regions can have sub-regions that are still membranes (presynaptic active zone). It is logically sound to say that any membrane that is a proper part of a membrane must be a membrane region. Reflexive part of may be better for inference in some cases, but I think we need to be v.careful about making such a big change to semantics so late in the game. Are you sure you want all subclasses of X to be inferred to be part_of some X? Obvious test to do: Is everyone happy with the new inferences we see if we make the switch experimentally in the editor's file? If the main problem is double-labelled edges in public-facing (pre-reasoned) releases, can't we just fix that with a simple rule-based system for stripping axioms (e.g. if both edges are asserted, retain only is_a)? |
Valid only under the interpretation that we haven't been using proper part of for years... |
On 17 Jan 2017, at 2:42, David Osumi-Sutherland wrote:
Hmmmm.
I'm fond of this pattern. I think it's pretty useful. Most biologists
are happy to take about membrane regions as membranes. Regions can
have some sub-regions that are still membranes. I think it is
completely sound to say that any membrane that is a proper part of a
membrane must be a membrane region.
we could keep the pattern without any commitment to reflexivity - just
have WM be a subclass of MR, and do away with the need for the MR
complement of WM.
Also, reflexive part of may be better for inference in some cases, but
I think we need to be v.careful about making such a big change to
semantics so late in the game. Are you sure you want all subclasses of
X to be inferred to be part_of some X? Obvious test to do: Is everyone
happy with the new we see if we make the switch experimentally in the
editor's file?
I agree about proceeding carefully (for OBA the axiom is spiked in for
reduction then removed). But I worry about building in assumptions about
part-of being proper.
If the main problem is double-labelled edges in public-facing
(pre-reasoned) releases, can't we just fix that with a simple
rule-based system for stripping axioms (e.g. if both edges are
asserted, retain only is_a)?
I think this may get increasingly ad hoc but onus on me to come up with
more examples
|
The fact that we are not resilient to this is a 'bad smell' IMO, I will
try and expand on this later.
As for advantages, the framework for handling redundancy becomes much
simpler. This will be a big advantage when we start using GCIs in DPs in
anger.
…On 17 Jan 2017, at 8:09, David Osumi-Sutherland wrote:
Switching to global reflexivity for PO is a v.big change this late in
the
game. We need clear doc of consequences if we make this move. Not
yet
clear to me what advantages are.
|
I added with the assumption of proper parthood - so wouldn't expect resilience. Just a matter of documenting modeling choices clearly.
Need more examples and some discussion.
I'd rather just do away with the 'whole membrane' terms. But then if you add reflexivity, you'll still have 'membrane region' equivalentTo membrane. |
What about this one:
We should at least put this in we have one assertion to it:
looks like all of the assertions should be inferred |
we have
If we add reflexivity of po we get unsats:
Note: I am using global reflexivity here because laziness and Elk doesn't support local reflexivity. Although there are issues with global reflexivity, as described here: https://github.com/oborel/obo-relations/wiki/ROGuide#local-reflexivity this is in fact unrelated to the issue at hand.
Why does this matter? Why not just treat part-of as proper part-of? I argue that part-of should be (locally) reflexive. The argument is partly from intuition (only an atom separates the two) but mostly from practicality. The practicality arises from situations where we want to use DOSDP GCIs to materialize non-redundant super class expressions. See for example obophenotype/bio-attribute-ontology#17
The text was updated successfully, but these errors were encountered: