Back to the main page.

Bug 531 - ft_freqanalysis does not work properly with mtmconvol and 'keeprpt' = 1

Status CLOSED FIXED
Reported 2011-03-15 14:09:00 +0100
Modified 2011-03-31 08:38:40 +0200
Product: FieldTrip
Component: specest
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P1 critical
Assigned to: Jan-Mathijs Schoffelen
URL:
Tags:
Depends on:
Blocks:
See also:

Jan-Mathijs Schoffelen - 2011-03-15 14:09:02 +0100

When input data has variable time axes across trials. Present functionality seems to be an cumulative sum over trials, including explicitly putting the 'nans' per trial to nan. This runs the risk of ending up with all nan data. e.g. when there are two trials in the data structure, one with the time axis running from -0.5 to 0 and the orther one with the time axis running from 0 to 0.5


Jan-Mathijs Schoffelen - 2011-03-15 15:51:52 +0100

I made a fix which I did not yet commit, because the adjusted version of ft_freqanalysis also contains new functionality to suppress output to the screen. This will only work properly if ft_progress is added to specest/private. Have to wait for Robert to explain to me how this should be done.


Jan-Mathijs Schoffelen - 2011-03-15 15:58:28 +0100

forget about my last remark; of course I can commit the change, because ft_progress is in utilities, which IS visible when the specest function is called by ft_freqanalysis (because this assumes utilities to be on the path). To keep specest as a separate module ft_progress does not need to be added to specest/private at all, because ft_progress will only be called if the caller function is ft_freqanalysis (same issue as above).