Confusing reference to ALL__INDICES

JimHokanson / SegwormMatlabClasses

Segworm code rewritten to use Matlab classes and packages

3 stars 0 forks source link

Confusing reference to ALL__INDICES #1

Closed MichaelCurrie closed 10 years ago

MichaelCurrie commented 10 years ago

@JimHokanson, in getPostureBends.m you have

SI          = seg_worm.skeleton_indices;
ALL_INDICES = SI.ALL_NORMAL_INDICES;

But in skeleton_indices.m you have already defined

ALL_INDICES = 1:49 %Same as 1:end,

Which is confusing. So in getPostureBends.m you should probably say ALL_NORMAL_INDICES = SI.ALL_NORMAL_INDICES, to avoid having that ALL_INDICES != SI.ALL_INDICES.

JimHokanson commented 10 years ago

Good point. I'll make the change for now. ALL_NORMAL_INDICES is really a poor name though. Alternative suggestions welcome.

JimHokanson commented 10 years ago

Changed the code ...