edickie / ciftify

The tools of the Human Connectome Project (HCP) adapted for working with non-HCP datasets
https://edickie.github.io/ciftify/
MIT License
112 stars 158 forks source link

Fixed test_utils.py and updated the patch for one test #70

Closed DESm1th closed 6 years ago

DESm1th commented 6 years ago

The test_utils.py (formerly test_utilities.py) test cases now work. The one modified test case inside test_ciftify_recon_all DOES NOT work, but the patch is now correct. That test is failing now because of the MSM feature you added contains a bug (or two)! If msm is not found config.find_msm() tries to call .replace on 'None'. Also config.verify_msm_available() tries to call sys.exit() but config.py hasn't imported sys.