Back to the main page.

Bug 2012 - ft_sourceinterpolate checks for 'pnt' field, uses 'pos' anyway

Status CLOSED FIXED
Reported 2013-02-27 15:49:00 +0100
Modified 2015-07-15 13:27:56 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P3 normal
Assigned to: Jim Herring
URL:
Tags:
Depends on:
Blocks:
See also:

Jim Herring - 2013-02-27 15:49:18 +0100

I am not sure if this has created problems but from line 205 in ft_sourceinterpolate the anatomical structure is checked for 'anatomical.pos' or 'anatomical.pnt' to fill a variable named 'pos'. However, even if 'anatomical.pnt' is found, 'pos' is still filled with 'anatomical.pos'. I imagine this will not work if only 'anatomical.pnt' exists. Is this deliberate? Code: anatomical = ft_checkdata(anatomical, 'hasunits', 'yes'); if isfield(anatomical, 'pos') pos = anatomical.pos; elseif isfield(anatomical, 'pnt') pos = anatomical.pos;


Jan-Mathijs Schoffelen - 2013-02-28 07:44:15 +0100

No, this is a bug, please go ahead and change the second occurrence of anatomical.pos into anatomical.pnt. Thanks for spotting this.


Jim Herring - 2013-02-28 09:12:49 +0100

Fixed and committed in rev. 7555