Skip to content

Conversation

byvernault
Copy link
Contributor

Fixes StatsCommand in Niftyseg interfaces.

Changes proposed in this pull request:

  • typo error line 78, it was UnaryStats instead of StatsCommand.
  • remove print statement in the StatsCommand.

Kind Regards,
Ben

@codecov-io
Copy link

Codecov Report

Merging #2106 into master will increase coverage by 23.6%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2106      +/-   ##
==========================================
+ Coverage   48.56%   72.17%   +23.6%     
==========================================
  Files         116     1144    +1028     
  Lines       23598    57604   +34006     
  Branches        0     8251    +8251     
==========================================
+ Hits        11461    41574   +30113     
- Misses      12137    14729    +2592     
- Partials        0     1301    +1301
Flag Coverage Δ
#smoketests 72.17% <0%> (+23.6%) ⬆️
#unittests 69.79% <0%> (?)
Impacted Files Coverage Δ
nipype/interfaces/niftyseg/stats.py 66.66% <0%> (ø)
nipype/interfaces/cmtk/parcellation.py 14.46% <0%> (ø)
...terfaces/freesurfer/tests/test_auto_Concatenate.py 85.71% <0%> (ø)
nipype/interfaces/tests/test_auto_SplitNifti.py 85.71% <0%> (ø)
nipype/utils/draw_gantt_chart.py 27.02% <0%> (ø)
...diffusion/tests/test_auto_gtractImageConformity.py 85.71% <0%> (ø)
...pe/interfaces/fsl/tests/test_auto_TractSkeleton.py 85.71% <0%> (ø)
nipype/interfaces/quickshear.py 100% <0%> (ø)
.../interfaces/fsl/tests/test_auto_PrepareFieldmap.py 85.71% <0%> (ø)
...ols/utilities/tests/test_auto_BRAINSEyeDetector.py 85.71% <0%> (ø)
... and 1084 more

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 af2b7aa...98f7351. Read the comment docs.

@effigies effigies merged commit 24fae18 into nipy:master Jul 10, 2017
@satra satra added this to the 0.14.0 milestone Oct 20, 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.

4 participants