Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ENH] FSL epi interface, add missing Eddy options. #1967

Merged
merged 1 commit into from
Apr 27, 2017
Merged

[ENH] FSL epi interface, add missing Eddy options. #1967

merged 1 commit into from
Apr 27, 2017

Conversation

imalone
Copy link

@imalone imalone commented Apr 24, 2017

A number of options in the most recent eddy tool were missing from the interface.

@imalone
Copy link
Author

imalone commented Apr 24, 2017

From what I can see the ci/circleci test failed because the docker image couldn't be found. I'm not able to re-run it.

@effigies
Copy link
Member

Triggered a rebuild. If you have your own CircleCI account (it's free), it should build under your user, which will give you the ability to re-trigger builds (and also your PRs won't be waiting on the repository queue).

@codecov-io
Copy link

codecov-io commented Apr 24, 2017

Codecov Report

Merging #1967 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1967      +/-   ##
==========================================
+ Coverage   72.52%   72.53%   +<.01%     
==========================================
  Files        1070     1070              
  Lines       54390    54397       +7     
  Branches     7858     7858              
==========================================
+ Hits        39448    39457       +9     
+ Misses      13715    13713       -2     
  Partials     1227     1227
Flag Coverage Δ
#smoketests 72.53% <100%> (ø) ⬆️
#unittests 70.08% <100%> (ø) ⬆️
Impacted Files Coverage Δ
nipype/interfaces/fsl/tests/test_auto_Eddy.py 85.71% <ø> (ø) ⬆️
nipype/interfaces/fsl/epi.py 66.15% <100%> (+0.61%) ⬆️
nipype/interfaces/base.py 84.84% <0%> (+0.18%) ⬆️

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 dc68922...2951749. Read the comment docs.

@imalone
Copy link
Author

imalone commented Apr 25, 2017

@effigies thanks. Useful to know about CircleCI, free stuff is always interesting...

@imalone
Copy link
Author

imalone commented Apr 27, 2017

Thanks!

@imalone imalone deleted the nipy/master branch April 27, 2017 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants