Skip to content

Conversation

@MichielCottaar
Copy link
Contributor

eddy_openmp is placed in the $FSLDIR/bin directory not the $FSLDIR directory

Fixes # 1.

Changes proposed in this pull request

  • Single minor bug-fix, which is needed to actually run eddy_openmp (if available) rather than eddy

I'm not sure how to test for this as the result depends on the content of $FSLDIR/bin

eddy_openmp is placed in the $FSLDIR/bin directory not the $FSLDIR directory
@codecov-io
Copy link

codecov-io commented Oct 24, 2017

Codecov Report

Merging #2249 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2249      +/-   ##
==========================================
- Coverage   72.32%   72.32%   -0.01%     
==========================================
  Files        1183     1183              
  Lines       59008    59008              
  Branches     8490     8490              
==========================================
- Hits        42679    42677       -2     
- Misses      14945    14947       +2     
  Partials     1384     1384
Flag Coverage Δ
#smoketests 72.32% <ø> (-0.01%) ⬇️
#unittests 69.91% <ø> (+0.01%) ⬆️
Impacted Files Coverage Δ
nipype/interfaces/fsl/epi.py 65.67% <ø> (ø) ⬆️
nipype/utils/profiler.py 40.36% <0%> (-1.21%) ⬇️

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 96c359d...2bb213d. Read the comment docs.

@satra satra merged commit 20b113c into nipy:master Oct 24, 2017
@satra satra modified the milestone: 0.14.0 Oct 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants