ENH: Set up customized report specification

[oesteban]

No description provided.

[codecov-io]

Codecov Report

Merging #34 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage   40.43%   40.37%   -0.07%     
==========================================
  Files           9        9              
  Lines         591      592       +1     
  Branches       93       93              
==========================================
  Hits          239      239              
- Misses        351      352       +1     
  Partials        1        1

Impacted Files	Coverage Δ
dmriprep/cli/run.py	28.91% <0%> (-0.11%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d7c8a6...3b71547. Read the comment docs.

[oesteban]

Great work @josephmje on this one!. I'll merge as soon as we get a new release of niworkflows (that's the reason for the one test failing).

[josephmje]

Great work @josephmje on this one!. I'll merge as soon as we get a new release of niworkflows (that's the reason for the one test failing).

I was just about to ask whether I should be concerned by the failed test. But looks like the anatomical workflows run now!

[josephmje]

With the new version of niworkflows, it looks like BIDSDataGrabberOutputSpec has changed to _BIDSDataGrabberOutputSpec in dmriprep/interfaces/__init__.py

[pull-assistant]

Best reviewed: commit by commit

---

Optimal code review plan (2 warnings)

initial reportlet creation

MANIFEST.in 50% changes removed in apply Oscar's sugges...

     minor fix

     use dmriprep reports module

     fix formatting

     remove duplicated code

apply Oscar's suggestions

MANIFEST.in 50% changes removed in pin(niworkflows): te...

dmriprep/config/report.tpl 43% changes removed in pin(niworkflows): te...

setup.cfg 50% changes removed in pin(niworkflows): te...

     pin(niworkflows): test out new report generation change and rebase

     Update setup.cfg

     fix: clean up nilearn

     Merge branch 'master' into enh/reportlet

Powered by Pull Assistant. Last update 46bee56 ... 2e8aadf. Read the comment docs.

[oesteban]

Okay, this seems to be working okay and the integration also works fine in fMRIPrep land - I'm going for it.