-
Notifications
You must be signed in to change notification settings - Fork 18
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
induced subtree request by id fails for genus Id #11
Comments
Seems like an OpenTree problem:
{
"message" : "Index: 32, Size: 32",
"exception" : "IndexOutOfBoundsException",
"fullname" : "java.lang.IndexOutOfBoundsException",
"stacktrace" : [ "java.util.ArrayList.rangeCheck(ArrayList.java:635)", "java.util.ArrayList.get(ArrayList.java:411)", "opentree.GraphExplorer.getDraftTreeMRCAForNodes(GraphExplorer.java:360)", "opentree.GraphExplorer.extractDraftSubtreeForTipNodes(GraphExplorer.java:468)", "opentree.plugins.tree_of_life.induced_subtree(tree_of_life.java:315)", "java.lang.reflect.Method.invoke(Method.java:606)", "org.neo4j.server.plugins.PluginMethod.invoke(PluginMethod.java:57)", "org.neo4j.server.plugins.PluginManager.invoke(PluginManager.java:168)", "org.neo4j.server.rest.web.ExtensionService.invokeGraphDatabaseExtension(ExtensionService.java:300)", "org.neo4j.server.rest.web.ExtensionService.invokeGraphDatabaseExtension(ExtensionService.java:122)", "java.lang.reflect.Method.invoke(Method.java:606)", "org.neo4j.server.rest.security.SecurityFilter.doFilter(SecurityFilter.java:112)" ]
} I hope to get this fixed today. |
Thanks! |
I've attempted to reproduce the issue, hoping to close this issue. Here's what I got:
Similar to previous result. However:
@josephwb Is this expected behavior? I was hoping that asking for subtree /w missing ott id would create subtree and perhaps indicate that some ott id was ignored because it wasn't in the draft tree. |
@jhpoelen No, this is not expected behaviour. A subtree should be returned, together with warnings if ids were 1) not in the synthetic tree or 2) altogether invalid ids. e.g. [edited, as there was a paste error in the curl call]: curl -X POST http://api.opentreeoflife.org/v2/tree_of_life/induced_subtree -H "content-type:application/json" -d '{"ott_ids":[357968, 867416, 939325, 9999999]}' {
"subtree" : "(Blastophaga_nipponica_ott939325,Blastophaga_psenes_ott357968,Blastophaga_quadrupes_ott867416)Blastophaga_ott462102;",
"ott_ids_not_in_tree" : [ ],
"ott_ids_not_in_graph" : [ 9999999 ],
"node_ids_not_in_graph" : [ ],
"node_ids_not_in_tree" : [ ]
} I believe what you are getting stems from an ongoing issue where one query node is the ancestor of another (can you confirm this? I can check, of course, but you probably know). Anyway, definitely not an rOTL issue. |
@josephwb from earlier example in this thread, the first three ott ids are tips (i.e. 357968, 867416, 939325) and the last one is a node (i.e. 462102) and wasp genus Blastophaga . Hope this helps. |
Ok, thanks. I have fixed this bug (not yet deployed), but am working on fixing another. |
hi @josephwb Did your fix get deployed? This still seems to fail on the OTL side (i.e., no subtree is being returned).
|
This works. |
Thanks for fixing this!
|
For some reason, induced subtree can be retrieved using ott ids c(357968, 867416, 939325) , but not c(357968, 867416, 939325, 462102) .
This succeeds:
but this fails:
Is this an opentree or rotl issue?
The text was updated successfully, but these errors were encountered: