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

Fix classes that reasoned into places too high in the hierarchy #1818

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

sebastianduesing
Copy link
Contributor

Adds subclass axioms to fix a handful of classes that were reasoning into places far too high in the hierarchy, per discussion with James.

James, earlier we talked about whether or not "pool of specimens" should be a subclass of specimen. I said no at the time, but after rereading its definition and the definition of specimen, I've changed my mind. I've put it as a subclass of specimen here. Let me know if you've got any concerns about that.

@jamesaoverton
Copy link
Contributor

Related to #1328.

I reviewed and tested this. I don't think it's controversial, and I want it for our OBI-COB work on PR #1812, so I'm going to merge it.

@jamesaoverton jamesaoverton merged commit 47a538e into master Aug 30, 2024
1 check passed
@jamesaoverton jamesaoverton deleted the fix_reasoned_hierarchy branch August 30, 2024 14:43
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