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