Back to the main page.

Bug 2695 - update documentation of ft_freqanalysis with method 'mtmconvol'

Status CLOSED FIXED
Reported 2014-09-22 13:16:00 +0200
Modified 2019-08-10 12:29:39 +0200
Product: FieldTrip
Component: documentation
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P5 normal
Assigned to: Tzvetan Popov
URL:
Tags:
Depends on:
Blocks:
See also:

Tzvetan Popov - 2014-09-22 13:16:34 +0200

when using cfg.method = 'mtmconvol' the configuration option cfg.foilim should be removed from the function help.


Eelke Spaak - 2014-09-22 13:21:16 +0200

Hmm sometimes I like using foilim, to be honest. I assume I missed a discussion about this during a meeting, sorry about that. But if anyone could briefly summarise the rationale for removing it, that would be great, thanks.


Tzvetan Popov - 2014-09-22 13:28:19 +0200

During the Birmingham WS it was stressed that cfg.foilim results in an error when used with the example data on the wiki. I'll try to reproduce that and report in a bit.


Tzvetan Popov - 2014-09-22 13:37:07 +0200

Eelke how would you specifiy cfg.t_ftimwin when cfg.foilim = [1 30] say? I think this is where the confusion came from, I think?


Eelke Spaak - 2014-09-22 13:38:22 +0200

(In reply to Tzvetan Popov from comment #3) Ah I'm sorry, I'm totally confused, I use foilim with mtmfft, not mtmconvol. Nevermind :)


Tzvetan Popov - 2014-09-22 13:40:36 +0200

(In reply to Eelke Spaak from comment #4) Yes right this is very convenient in fft ;-)


Tzvetan Popov - 2014-09-22 13:46:19 +0200

bash-4.1$ svn commit ft_freqanalysis.m -m 'documentation- updated the doc when using mtmconvol see bug2695' Sending ft_freqanalysis.m Transmitting file data . Committed revision 9820.


Robert Oostenveld - 2019-08-10 12:29:39 +0200

This closes a whole series of bugs that have been resolved (either FIXED/WONTFIX/INVALID) for quite some time. If you disagree, please file a new issue describing the issue on https://github.com/fieldtrip/fieldtrip/issues.