Upgrade to owlapy 1.6.2 #557
Merged
+139
−236
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
Thank you for the PR. I will review it tomorrow:) |
…1.6.2 # Conflicts: # ontolearn/concept_learner.py
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades the project to owlapy 1.6.2 and dicee 0.2.0, refactoring code to use the new API patterns. The neural reasoner serving script is moved to the owlapy package, and the README is improved for clarity.
Key changes:
- Replace
TripleStoreNeuralReasonerwithEBR(NeuralOntology(...))pattern throughout the codebase - Update imports from ontolearn to owlapy for neural reasoning functionality
- Upgrade dependency versions and relax torch version constraint from
==2.2.0to>=2.2.0
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| setup.py | Updates dependency versions: owlapy 1.5.1→1.6.2, dicee 0.1.4→0.2.0, torch constraint relaxed |
| ontolearn/triple_store.py | Fixes typo: reminder → remainder for IRI property access |
| ontolearn/semantic_caching.py | Migrates from TripleStoreNeuralReasoner to EBR(NeuralOntology(...)) |
| ontolearn/scripts/litserve_neural_reasoner.py | Removes entire file (moved to owlapy) |
| ontolearn/learners/nces2.py | Improves file handling with absolute paths and mode 'w' instead of 'a' |
| ontolearn/learners/nces.py | Updates dicee version, improves file handling, adds debugging output |
| ontolearn/learners/celoe.py | Renames import: OperandSetTransform → CESimplifier |
| ontolearn/learners/base.py | Extends type checking to include TripleStoreOntology |
| examples/concept_learning_neural_evaluation.py | Migrates to new neural reasoner API |
| tests/test_example_concept_learning_neural_evaluation.py | Updates test code (commented) with new API |
| tests/test_base_concept_learner.py | Removes unused tempfile import |
| README.md | Improves webservice documentation with clearer structure and examples |
| .coveragerc | Adds explicit source directive for coverage configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
Author
|
@Demirrr ready for your review |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
owlapy 1.6.2anddicee 0.2.0ontolearn/scripts/litserve_neural_reasoner.pyis moved to owlapy