Skip to content
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

study_doi should be replaced by studyPublicationReference #105

Open
bomeara opened this issue Dec 7, 2017 · 1 comment
Open

study_doi should be replaced by studyPublicationReference #105

bomeara opened this issue Dec 7, 2017 · 1 comment

Comments

@bomeara
Copy link
Contributor

bomeara commented Dec 7, 2017

The documentation says study_doi is supported, but this seems to have been changed to studyPublication by OT.

studies_find_studies(property="ot:study_doi", value="http://dx.doi.org/10.1600/036364411X605092")

leads to

Error: HTTP failure: 400
unrecognized property: ot:study_doiBadInputExceptionorg.neo4j.server.rest.repr.BadInputExceptionlist("org.opentree.oti.plugins.studies_v3.find_studies(studies_v3.java:100)", "java.lang.reflect.Method.invoke(Method.java:498)", "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:498)", 
    "org.neo4j.server.rest.security.SecurityFilter.doFilter(SecurityFilter.java:112)")

whereas

studies_find_studies(property="ot:studyPublication", value="http://dx.doi.org/10.1600/036364411X605092")

leads to

  study_ids n_trees                     tree_ids candidate study_year                                                                                title                                  study_doi
1    pg_719       3 tree1294, tree1295, tree1296  tree1296       2011 Phylogenetic relationships and morphological evolution in Nymphoides (Menyanthaceae) http://dx.doi.org/10.1600/036364411X605092

This affects studies_find_studies, but I think also studies_find_trees. I'm using rotl 3.0.3.

@fmichonneau
Copy link
Member

Hi @bomeara

I can't seem to find a reference to study_doi in the context of the properties that can be used to search for trees/studies. I think study_doi is only our name for the column of the table returned by studies_find_studies and studies_find_trees. It can be that the documentation needs to be made clearer!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants