ofgulban / segmentator

3D MRI data exploration and segmentation tool.
https://github.com/ofgulban/segmentator/wiki
BSD 3-Clause "New" or "Revised" License
74 stars 13 forks source link

Removed redundant sys argument retrieval. #57

Closed ingo-m closed 7 years ago

ingo-m commented 7 years ago

I removed the redundant sys argument retrieval. The input arguments were retrieved twice, once using sys, and once using argparse. The list (args) retrieved with sys was subsequently overwritten by a namespace object from argparse. This is just a suggestion, but I think it is more pythonic not include redundant code.

codecov-io commented 7 years ago

Codecov Report

Merging #57 into devel will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           devel     #57   +/-   ##
=====================================
  Coverage   3.65%   3.65%           
=====================================
  Files          6       6           
  Lines        847     847           
=====================================
  Hits          31      31           
  Misses       816     816

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 b5ed056...6bb41bf. Read the comment docs.