Skip to content

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