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

Changed default value of mni template #1109

Merged
merged 1 commit into from Aug 12, 2016

Conversation

Projects
None yet
5 participants
@riddhishb
Contributor

riddhishb commented Aug 11, 2016

In reference to @jchoude comment on #1106, I have changed the default value back to T2

@coveralls

This comment has been minimized.

coveralls commented Aug 11, 2016

Coverage Status

Coverage remained the same at 83.025% when pulling 45b791f on riddhishb:mni_fix into b64971b on nipy:master.

@codecov-io

This comment has been minimized.

codecov-io commented Aug 11, 2016

Current coverage is 80.96% (diff: 100%)

Merging #1109 into master will not change coverage

@@             master      #1109   diff @@
==========================================
  Files           205        205          
  Lines         23329      23329          
  Methods           0          0          
  Messages          0          0          
  Branches       2341       2341          
==========================================
  Hits          18889      18889          
  Misses         3958       3958          
  Partials        482        482          

Powered by Codecov. Last update b64971b...45b791f

@RafaelNH

This comment has been minimized.

Contributor

RafaelNH commented Aug 11, 2016

If no one opposes, I will merge this at the end of this day!

@jchoude

This comment has been minimized.

Contributor

jchoude commented Aug 11, 2016

Seems good to me. Thanks :)

@RafaelNH RafaelNH merged commit 31bc6fe into nipy:master Aug 12, 2016

4 checks passed

codecov/patch 100% of diff hit (target 80.96%)
Details
codecov/project 80.96% (+0.00%) compared to b64971b
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 83.025%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment