Back to the main page.

Bug 1268 - statfun_indepsamplesZcoh needs cfg.label or cfg.pos in input, but they are not used

Status CLOSED FIXED
Reported 2012-01-19 09:08:00 +0100
Modified 2012-03-21 15:45:10 +0100
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P3 normal
Assigned to: Jan-Mathijs Schoffelen
URL:
Tags:
Depends on:
Blocks:
See also:

Jan-Mathijs Schoffelen - 2012-01-19 09:08:03 +0100

only to determine the number of channels/voxels ft_freqstatistics crashes when not providing cfg.label in the input. Apart from the fact that this is an inconsistent name (should be channel), it suggests as if something can be configured. However, all channel pairs are always computed. The option cfg.pos/cfg.label should be removed or properly implemented; if it is implemented change the name to cfg.channel. I suggest to remove it. Hardly anybody uses this function anyway. For a test, see test_bug895


Jan-Mathijs Schoffelen - 2012-01-19 09:46:07 +0100

fixed in revision 5149, see also bug 895 and corresponding test_bug895