Back to the main page.

Bug 999 - sourceinterpolates sets default projectmethod to 'linear' while such a method does not exist

Status CLOSED DUPLICATE
Reported 2011-10-03 14:54:00 +0200
Modified 2011-11-09 14:37:42 +0100
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P1 normal
Assigned to: Jörn M. Horschig
URL:
Tags:
Depends on:
Blocks:
See also:

Jörn M. Horschig - 2011-10-03 14:54:58 +0200

hooray, one thing works, next fails ;) In ft_sourceinterpolate, interp_ungridded is called with 'projectmethod' = 'linear' by default (line 116 and line 195), but interp_ungridded does not support 'linear' as a method (as also the help text suggests, it should be 'nearest', 'smudge or 'sphere_avg'. This part is also missing in the FT tutorial. What method to use by default? btw, #999, one more!


Jörn M. Horschig - 2011-10-05 10:20:27 +0200

uh, the bugreporter asked a really good question here. I do not know the answer, do you, JM? :)


Jörn M. Horschig - 2011-10-14 13:04:40 +0200

hey JM, set the default to 'nearest' and commited, please check whether that's alrighty


Jan-Mathijs Schoffelen - 2011-10-14 13:11:33 +0200

Hi Jörn, No, probably not OK. I think the default should still be linear (as this has been the default for the past 7 or so years). It's interp_ungridded who misbehaves. If the traditional code is used, 'linear' as method is supported. We should either make the default value conditional on the is2Dfun/is2Dana etc, or make interp_ungridded support 'linear' as an interpolation method.


Jörn M. Horschig - 2011-10-14 16:06:49 +0200

good I'll change it back and commit again


Jörn M. Horschig - 2011-11-09 09:38:18 +0100

*** This bug has been marked as a duplicate of bug 1103 ***