search

INCF / BrainImagingPipelines

BIPs
Other
28 stars 31 forks source link

Fix for QA_utils.py #74

Closed mrpeverill closed 10 years ago

mrpeverill commented 10 years ago

QA_utils.py crashed in roistripper with "too many indices" when summary file had comments or when roi_file had only one row. Corrected by built-in numpy comment stripping and forcing the roi array to at least a two dimensional array.

satra commented 10 years ago

@mrpeverill - the key here is only to fix when the roi file has one row. could you please revise this request to only make the necessary change.

# comments are automatically taken care of by genfromtxt. also your editor is inserting tabs instead of using spaces. most editors today will have an option to use spaces instead of tabs.

mrpeverill commented 10 years ago

Oh sorry - I made the fixes in order, so I didn't realize the first didn't do me any good. I made a new commit for just the necessary change (and I fixed the whitespace to be consistent with master)