Skip to content

Turn on ROBOT's multiple_equivalent_class_definitions check - #26

Open
hoodcm wants to merge 1 commit into
mainfrom
robot-multiple-equivalent-class-definitions
Open

hoodcm wants to merge 1 commit into
mainfrom
robot-multiple-equivalent-class-definitions

Conversation

@hoodcm

@hoodcm hoodcm commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Part of #5.

Check Data checked Why
multiple_equivalent_class_definitions Logical axioms A class has at most one logical definition.

ROBOT's standard check, at its default level (ERROR). Nothing on main fails it, so no data changes. All tests pass.

@jclerman jclerman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would not merge this. Multiple logical definitions on a single class can actually be quite informative, e.g.

left knee
== knee and (part_of some left leg)
== knee and (part of some left side of body)

The argument for this rule in the ROBOT docs is vague (to me), and I've found considerable utility in the kinds of constructions it prohibits.

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.

2 participants