-
Notifications
You must be signed in to change notification settings - Fork 8
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
Conversion to neo lite and golr #260 #261
Conversation
Should not be possible when launching minerva normally.
List<GolrOntologyClassDocument> ontologyEntities = ontologyClient.getGolrOntologyCls(curie); | ||
if (ontologyEntities != null && !ontologyEntities.isEmpty()) { | ||
result = new ArrayList<ExternalLookupService.LookupEntry>(ontologyEntities.size()); | ||
for(GolrOntologyClassDocument doc : ontologyEntities) { | ||
result.add(new LookupEntry(id, doc.annotation_class_label, "ontology_class", null)); | ||
result.add(new LookupEntry(id, doc.annotation_class_label, "ontology_class", doc.only_in_taxon, doc.isa_partof_closure)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goodb shouldn't this be only isa_closure
, not isa_partof_closure
?
@@ -58,7 +58,7 @@ protected MonarchExternalLookupService(String golrUrl, RetrieveGolrOntologyClass | |||
if (entities != null && !entities.isEmpty()) { | |||
result = new ArrayList<ExternalLookupService.LookupEntry>(entities.size()); | |||
for(GolrOntologyClassDocument doc : entities) { | |||
result.add(new LookupEntry(id, doc.annotation_class_label, "ontology_class", null)); | |||
result.add(new LookupEntry(id, doc.annotation_class_label, "ontology_class", null, doc.isa_partof_closure)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goodb shouldn't this be only isa_closure
, not isa_partof_closure
? (same question as in other file)
Yes, I think it should be just isa_closure. The OWLTools class GolrOntologyClassDocument that minerva was using doesn't have just the isa_closure element. I think for what we are doing it won't really matter, but you are right. I will go ahead and add it in to my stolen instance. Hopefully GOLR supports that query properly. |
No description provided.