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

Add reciprocal_of in inverse direction. #243

Merged
merged 1 commit into from
Jul 2, 2019

Conversation

balhoff
Copy link
Contributor

@balhoff balhoff commented May 20, 2019

This is useful for querying; can't be inferred at class level.

@drseb
Copy link
Member

drseb commented Jun 4, 2019

Can you explain to me the difference between is_opposite_of and reciprocal_of ?

@balhoff
Copy link
Contributor Author

balhoff commented Jun 5, 2019

I don't think reciprocals are opposites. Basically it's just an inversion of the EQ model between inheres in and towards. So:

absent and 'inheres in' some eye

is reciprocal of:

'lacks all parts of type' and towards some 'eye'

@wdahdul wdahdul self-requested a review June 27, 2019 21:01
@wdahdul
Copy link
Contributor

wdahdul commented Jun 27, 2019

Adding an approval on this one - will be needed for Phenoscape queries on absence

@balhoff
Copy link
Contributor Author

balhoff commented Jul 2, 2019

@nicolevasilevsky could you merge this?

@nicolevasilevsky
Copy link
Contributor

@wdahdul I think you need to officially approve this, it says merging is blocked, Merging can be performed automatically with 2 approving reviews.

@nicolevasilevsky nicolevasilevsky merged commit fa81da6 into pato-ontology:master Jul 2, 2019
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.

5 participants