Back to the main page.

Bug 220 - freqanalysis: if cfg.foi contains frequencies above Nyqiest an error should be given

Status CLOSED FIXED
Reported 2010-11-15 17:17:00 +0100
Modified 2010-12-08 11:55:40 +0100
Product: FieldTrip
Component: core
Version: unspecified
Hardware: All
Operating System: Windows
Importance: P1 normal
Assigned to: Roemer van der Meij
URL:
Tags:
Depends on:
Blocks:
See also:

Roemer van der Meij - 2010-11-15 17:17:22 +0100

As the Nyquist is a hard-limit on the frequencies one can estimated, an error should be thrown when a user accidentally requests it. This sounds like a no-brainer not to do, but if your downsampling a lot mistakes can be made much easier. Someone came to me today who had down-sampled MEG data to 100Hz and forgot about it (as it did not matter at the time), and accidentally estimated up until 70 Hz. It should be a really easy check in an if-construct.


Roemer van der Meij - 2010-11-15 18:18:18 +0100

Ouch, my English is apparently not the best these days! Luckily one can read through the errors and get the gist.


Roemer van der Meij - 2010-12-08 11:55:32 +0100

Added an error for mtmconvol and mtmfft separately. As soon as specest is fully implemented, these will need to moved/combined.