ENH: Add number of volumes per *b*-value to QC report

[josephmje]

Closes #64 and #128. Replaces PR #73.

• [ ] report diffusion sampling scheme: shelled, radial or Cartesian
• [x] report shell distribution
• [x] expose bmag parameter to adjust rounding bvals
• [x] check # b0s doesn't change after rounding
• [ ] create tests for DiffusionSummary interface

In the case of DSI data, should round_bvals automatically round to the nearest 10 instead of calculating based on the magnitude of bvals? Set bmag = 1.

[oesteban]

Could you rebase when #151 is merged?

[josephmje]

I'll update this to see if I can figure out a parser that determines whether the data is shelled or sampled along the Cartesian grid. I've checked out the methods section of a few DSI papers and they seem to either report the maximum b value or a range (eg. x DWIs with diffusion weightings in the range of b = 100-4000 s/mm2 and x interleaved b = 0 scans). Perhaps a similar approach can be taken in our reportlet.

[oesteban]

I'll update this to see if I can figure out a parser that determines whether the data is shelled or sampled along the Cartesian grid. I've checked out the methods section of a few DSI papers and they seem to either report the maximum b value or a range (eg. x DWIs with diffusion weightings in the range of b = 100-4000 s/mm2 and x interleaved b = 0 scans). Perhaps a similar approach can be taken in our reportlet.

Yes, this sounds fantastic.

[pep8speaks]

Hello @josephmje, Thank you for updating!

Cheers! There are no style issues detected in this Pull Request. :beers: To test for issues locally, pip install flake8 and then run flake8 dmriprep.

Comment last updated at 2021-03-11 15:15:14 UTC

[josephmje]

This is ready for review. I will leave the DSI updates for another PR.

[josephmje]

@oesteban tests are failing after some of the changes to the dwi reference workflow. will wait until #153 gets merged and then will rebase

[oesteban]

sounds great

[oesteban]

should we resuscitate this?