Back to the main page.

Bug 2289 - Filters don't work when doing preprocessing

Status CLOSED INVALID
Reported 2013-09-16 13:00:00 +0200
Modified 2019-08-10 12:03:57 +0200
Product: FieldTrip
Component: preproc
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P3 normal
Assigned to:
URL:
Tags:
Depends on:
Blocks:
See also:

- 2013-09-16 13:00:37 +0200

When I do this the signal isn't filtererd cfg.preproc.demean = 'yes'; cfg.preproc.baselinewindow = [-0.2 0]; cfg.preproc.reref = 'yes'; cfg.preproc.refchannel = {'EEG'}; cfg.preproc.lpfilter = 'yes'; cfg.preproc.lpfreq = 30; % dataPE = ft_preprocessing(cfg); but when I remove the preproc part, it works but gives me a warning cfg.demean = 'yes'; cfg.baselinewindow = [-0.2 0]; cfg.reref = 'yes'; cfg.refchannel = {'EEG'}; cfg.lpfilter = 'yes'; cfg.lpfreq = 30; % dataPE = ft_preprocessing(cfg);


Robert Oostenveld - 2013-09-16 13:27:12 +0200

according to the documentation of ft_preprocessing your first cfg specification is not correct, but the second is. Your first specification is used in some functions (e.g. ft_timelockanalysis) that do some preprocessing as optional auxiliary step. What is the warning you get?


- 2013-09-16 14:29:07 +0200

I forgot to mention that I get the warning when I use ft_databrowser and not ft_preprocessing. Warning: The field cfg.reref is deprecated, please specify it as cfg.preproc.reref instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.refchannel is deprecated, please specify it as cfg.preproc.refchannel instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.demean is deprecated, please specify it as cfg.preproc.demean instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.baselinewindow is deprecated, please specify it as cfg.preproc.baselinewindow instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.dftfilter is deprecated, please specify it as cfg.preproc.dftfilter instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.dftfreq is deprecated, please specify it as cfg.preproc.dftfreq instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.hpfilter is deprecated, please specify it as cfg.preproc.hpfilter instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.hpfreq is deprecated, please specify it as cfg.preproc.hpfreq instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.lpfilter is deprecated, please specify it as cfg.preproc.lpfilter instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124 Warning: The field cfg.lpfreq is deprecated, please specify it as cfg.preproc.lpfreq instead of cfg. > In ft_checkconfig at 445 In ft_databrowser at 124


Robert Oostenveld - 2013-09-16 14:50:13 +0200

(In reply to comment #2) in databrowser you should use cfg.preproc. So in short please use cfg.preproc everywhere, except for ft_preprocessing.


Robert Oostenveld - 2019-08-10 12:03:57 +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.