-
Notifications
You must be signed in to change notification settings - Fork 2
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
Remove Caching #97
Comments
Caching can be disabled or enabled by the user. If you are referring to the automatic disable of caching when ontology modification is detected then that's something that is done to protect the user without him having to do the job manually because if axioms are added or removed from the ontology where the reason methods are called then in order to avoid inconsistencies with the information stored in cache we need to disable it. |
These lines show that we cannot disable right ?! |
Okay there seems to be a misunderstanding, I thought we were talking about class and property cache. Its true that the LRU caches cannot be disabled, but since they are now gone #100 there is no cache left that cant be disabled. |
We should not add any functionalities that cannot be disabled by the user. StructuralReasoner must be refactored!
The text was updated successfully, but these errors were encountered: