dprice80 / mni2fs

MNI2FS: Surface Rendering of MNI Space Volumes for MATLAB
GNU General Public License v3.0
15 stars 6 forks source link

Undefined variable 'curv' when decimation is set to 0 #10

Closed cmicek1 closed 6 years ago

cmicek1 commented 6 years ago

Great package Darren! I just downloaded it and started playing around with it today, and noticed that when setting the FreeSurfer decimation flag to 0, there is a variable called 'curv' that is undefined in mni2fs_brain.m. It would appear that higher-resolution vertex locations are absent from the repository in the 'surf' folder; if these exist could you update/add accordingly? Seems like a pretty quick fix to me.

cmicek1 commented 6 years ago

False alarm; apparently that's only a problem when modifying an existing struct

dprice80 commented 6 years ago

Ah yes that would do it. Thanks for letting me know anyway.

On Thu, 7 Jun 2018, 18:26 Christopher Micek, notifications@github.com wrote:

False alarm; apparently that's only a problem when modifying an existing struct

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dprice80/mni2fs/issues/10#issuecomment-395501035, or mute the thread https://github.com/notifications/unsubscribe-auth/AHtKBkgX5Hk2GUjT25ribjl-MlK8U2xyks5t6WI4gaJpZM4UevOX .

dprice80 commented 6 years ago

There is a new object oriented version cooking soon with many new features

On Thu, 7 Jun 2018, 18:26 Christopher Micek, notifications@github.com wrote:

False alarm; apparently that's only a problem when modifying an existing struct

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dprice80/mni2fs/issues/10#issuecomment-395501035, or mute the thread https://github.com/notifications/unsubscribe-auth/AHtKBkgX5Hk2GUjT25ribjl-MlK8U2xyks5t6WI4gaJpZM4UevOX .