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

added disjoint property axioms #33

Closed
wants to merge 1 commit into from
Closed

Conversation

mathib
Copy link
Member

@mathib mathib commented Jun 15, 2018

As discussed in #24 a disjoint property axiom is added for bot:containsElement and bot:adjacentElement. A similar axiom was added for bot:containsZone and bot:adjacentZone. All four the definitions of the properties are updated: either bot:containsElement/Zone or bot:adjacentElement/Zone has to be used, but not both.

The main reason behind this restriction, is that a bot:containsZone / bot:containsElement relation means that the complete bot:Zone / bot:Element has to be contained in the other bot:Zone, instead of only partially.

@deKlerk
Copy link

deKlerk commented Jun 21, 2019

LDAC note:
The pull request needs to be updated so that the newer intersection property is also disjoint with the other properties. It needs to be compatible with the newer BOT ontology. Otherwise, there is a tendency to accept the pull request.

@GeorgFerdinandSchneider
Copy link
Collaborator

This PR seems to be outdated. Issue #24 has been reopened to find a final decision on this topic.

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.

None yet

3 participants