niworkflows icon indicating copy to clipboard operation
niworkflows copied to clipboard

ENH: add gradunwarp base workflow for f/smriprep.

Open bpinsard opened this issue 1 year ago • 3 comments

Working on https://github.com/nipreps/fmriprep/issues/1550. Relies on an updated version of the gradunwarp https://github.com/bpinsard/gradunwarp/tree/fix/mem_leaks.

bpinsard avatar Aug 02 '23 13:08 bpinsard

Codecov Report

Attention: Patch coverage is 48.48485% with 34 lines in your changes are missing coverage. Please review.

Project coverage is 62.96%. Comparing base (a9f00d5) to head (2ec6bfc).

:exclamation: Current head 2ec6bfc differs from pull request most recent head 2c1507a

Please upload reports for the commit 2c1507a to get more accurate results.

Files Patch % Lines
niworkflows/workflows/gradunwarp.py 18.51% 22 Missing :warning:
niworkflows/interfaces/gradunwarp.py 67.56% 12 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #819      +/-   ##
==========================================
- Coverage   68.52%   62.96%   -5.57%     
==========================================
  Files          86       55      -31     
  Lines        8220     6316    -1904     
  Branches      949      914      -35     
==========================================
- Hits         5633     3977    -1656     
+ Misses       2580     2152     -428     
- Partials        7      187     +180     
Flag Coverage Δ
reportlettests ∅ <ø> (?)
unittests ∅ <ø> (?)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Aug 02 '23 14:08 codecov[bot]

I'm guessing if you want circle / gh to run the test, you'll need to install gradunwarp right?

oesteban avatar Aug 07 '23 11:08 oesteban

I don't understand why I am getting errors on the T1w file when testing the gradunwarp pipeline. How does that ds0000030 differs from the openeuro one?

bpinsard avatar May 16 '24 19:05 bpinsard