Back to the main page.
Bug 2189 - freqanalysis with cfg.keeptrials='no' keeps cumtapcnt
Status | CLOSED FIXED |
Reported | 2013-06-06 10:34:00 +0200 |
Modified | 2014-01-29 13:28:42 +0100 |
Product: | FieldTrip |
Component: | core |
Version: | unspecified |
Hardware: | PC |
Operating System: | Windows |
Importance: | P3 minor |
Assigned to: | Eelke Spaak |
URL: | |
Tags: | |
Depends on: | |
Blocks: | |
See also: |
Jörn M. Horschig - 2013-06-06 10:34:58 +0200
cumtapcnt refers to the trials, so no reason to keep cumtapcnt if cfg.keeptrials = 'no': example cfgf = []; cfgf.foi = 1:2:128; cfgf.method = 'mtmconvol'; cfgf.taper = 'hanning'; cfgf.toi = -1.2:.025:.5; cfgf.t_ftimwin = 0.5 * ones(size(cfgf.foi))'; cfgf.keeptrials = 'no'; freq= ft_freqanalysis(cfgf, data) ans = label: {7x1 cell} dimord: 'chan_freq_time' freq: [1x64 double] time: [1x69 double] powspctrm: [7x64x69 double] cumtapcnt: [429x64 double] cfg: [1x1 struct]
Eelke Spaak - 2013-10-09 12:13:16 +0200
fixed in 8590 (used the solution that was already there for cumsumcnt).
Jan-Mathijs Schoffelen - 2013-11-03 21:48:35 +0100
I think that test_ft_freqanalysis fails because the stored testdata still has the cumtapcnt. Therefore, I reopen this one to remember to upload a new version of the frequency domain test data. Yet, the test script throws an error because the identical function (in test/private) returns a cell-array, but the test-function expects it to be a string.
Eelke Spaak - 2013-11-04 09:30:40 +0100
test script fixed in rev 8711. However, when trying to update the stored data (writeflag = 1), I get the following error: Error using save Unable to write file /home/common/matlab/fieldtrip/data/test/latest/freq/meg/freq_mtmfft_powandcsd_trl_bti248: permission denied. the rest of the data files (or at least the ones before that) seem to work fine.
Robert Oostenveld - 2013-11-04 09:35:02 +0100
(In reply to comment #3) some files in that directory had permissions rw-r--r--, rather than rw-rw-r--. I fixed the permissions for all files in /home/common/matlab/fieldtrip/data/test/latest, they can now be overwritten.
Eelke Spaak - 2013-11-04 11:01:07 +0100
fixed, overwritten all test data