search

elixir-europe / plant-brapi-to-isa

BSD 3-Clause "New" or "Revised" License
8 stars 6 forks source link

Documentation: test endpoints returning an error due to wrong Trial or Study Id as input #60

Closed proccaserra closed 5 years ago

proccaserra commented 5 years ago

on URGI server, Trial and Study Ids no longer have numerical values for example, they should be looking like this dXJuOlVSR0kvc3R1ZHkvQlRIX09yZ2V2YWxfMjAwMl9TZXRBMg==

so documentation should reflect this for the conversion to run(*): python brapi_to_isa.py -e https://urgi.versailles.inra.fr/faidare/brapi/v1/ -s dXJuOlVSR0kvc3R1ZHkvQlRIX09yZ2V2YWxfMjAwMl9TZXRBMg==

(*) I am also running into error when converting the Trial associated with 'POPYOMICS' program (related to https://github.com/ISA-tools/isa-api/issues/344 as I am trying to reproduce the error'

bedroesb commented 5 years ago

Good catch. I already use the new ID's in the CI, but forgot to change the documentation.

bedroesb commented 5 years ago

fixed in #59