Fix #3149 #3152
Fix #3149 #3152
Conversation
| buffer.seek(0) | ||
|
|
||
| # validate on load | ||
| data = qiita_db.metadata_template.util.load_template_to_dataframe(rawdata) |
There was a problem hiding this comment.
I should have been clearer, sorry. This needs to be imported, add at the top:
from qiita_db.metadata_template.util import load_template_to_dataframe
| data = pd.DataFrame.from_dict(json_decode(self.request.body), | ||
| orient='index') | ||
| # convert from json into a format that qiita can validate | ||
| rawdata = pd.DataFrame.from_dict(json_decode(self.request.body), |
There was a problem hiding this comment.
I think the load_template_to_dataframe accepts string so perhaps worth not converting to dataframe and then back and just do:
data = load_template_to_dataframe(json_decode(self.request.body)) but I'm not sure if json_decode returns a string or a byte.
There was a problem hiding this comment.
The code looks like it opens a filehandle, so I don't think a regular string would work. The spec for the REST endpoint as well is not a str representation of a TSV, but a JSON representation derived from DataFrame.to_json. I may be missing something, but I think this transform is necessary
add qiita_db/studies/ endpoint
...route load through Qiita's template loading mechanisms