Codecov Report

Merging #645 (3a3d837) into maint/1.3.x (691436a) will decrease coverage by 47.95%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           maint/1.3.x    #645       +/-   ##
===============================================
- Coverage        47.95%   0.00%   -47.96%     
===============================================
  Files               43      43               
  Lines             5303    5301        -2     
  Branches           775     775               
===============================================
- Hits              2543       0     -2543     
- Misses            2673    5301     +2628     
+ Partials            87       0       -87     

Flags with carried forward coverage won't be shown. Click here to find out more.

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 691436a...3a3d837. Read the comment docs.

I think the keyword is delimiter? I can't open the failing test on my phone, though.

As far as I know, pandas supports both the sep and delimiter keyword arguments.

I just had a look, the error is something else.

E   ImportError: A sklearn version of at least 0.21 is required to use nilearn. 0.19.2 was found. Please upgrade sklearn

Got it. So we need to update our Dockerfile around here: https://github.com/nipreps/niworkflows/blob/master/Dockerfile#L126

Should I submit a separate PR for the Dockerfile update?

I think we can use this PR :)