-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add has_subClasses flag #43
Comments
For this to work, we need pre-reasoning to happen before the SPARQL. This is not currently the case. SPARQL runs on the triplestore. It would work if we ran SPARQL construct using ROBOT after this stage: Also need to check whether product of construct_all.owl ends up in both Neo & SOLR (looks like it goes into SOLR via obographs.json goal. CC @hkir-dev |
Neo4j2owl is processing these labels as well: https://github.com/VirtualFlyBrain/neo4j2owl/blob/migrate_neo4j_hk/src/main/java/ebi/spot/neo4j2owl/importer/N2OOntologyLoader.java#L381 I can start the implementation of ROBOT based construct files processing. I don't think it will cause any performance issues. Ps: We should be careful about memory usage since ROBOT query is using jena afaik and reason is using owlapi. If we combine these functions there is a risk to use 2X memory. But it won't be a problem if we use separate processes to run these functions. |
New step should run after reasoning but before merge. |
Requires pipeline modification specified here: #48 |
#48 has now been merged |
Use case - driving query display
The text was updated successfully, but these errors were encountered: