PennLINC / qsiprep

Preprocessing of diffusion MRI
http://qsiprep.readthedocs.io
BSD 3-Clause "New" or "Revised" License
138 stars 55 forks source link

[ENH] Write a report.json derivative that can be loaded in dmriprep-viewer #165

Closed mattcieslak closed 8 months ago

mattcieslak commented 4 years ago

Save this along with the other figures in the derivatives

Tagging @richford

mattcieslak commented 4 years ago

Can add a javascript button that will go somwhere here: https://github.com/PennBBL/qsiprep/blob/master/qsiprep/viz/report.tpl#L174

richford commented 4 years ago

In addition to the content already in the example json, the json should include the following fields:

richford commented 4 years ago

Also, after the conversation in our telecon this morning, perhaps we should add a json sidecar to report.json the same way we do for nifti files.

richford commented 4 years ago

I've made a public S3 bucket at s3://dmriprep-viewer-example-data with a sample layout for these reports. Please use this as a reference spec for the json reports. Things to note:

richford commented 4 years ago

Tagging @arokem for visibility of the json report spec.

richford commented 4 years ago

BTW, pandas.DataFrame.to_json(orient="records") should create the subjects array in exactly the format that I've specified.

arokem commented 4 years ago

It's fine for this to be a public S3 bucket, if there's good reason. As an alternative, consider using OSF's storage.

richford commented 4 years ago

@arokem, good point. I wanted to include S3 support in dmriprep-viewer so that we could support HBN and any dataset on OpenNeuro. But I opened an issue on dmriprep-viewer about OSF support.