Back to the main page.

Bug 962 - remove default cfg.demean = 'yes' in ft_componentanalysis

Status CLOSED WONTFIX
Reported 2011-09-14 15:15:00 +0200
Modified 2019-08-10 12:03:38 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P1 normal
Assigned to:
URL:
Tags:
Depends on:
Blocks:
See also:

Stephen Whitmarsh - 2011-09-14 15:15:41 +0200

It really doesnt make sense anymore. Everyone agreed?


Jan-Mathijs Schoffelen - 2011-10-03 13:17:50 +0200

why?


Stephen Whitmarsh - 2011-10-03 13:26:21 +0200

1) its an 'active' default, i.e. it does things you would not expect without you asking. 2) it does not make sense to demean before you do ICA for artifact removal where large differences in offset can be expected due to e.g. eyeblinks.


Jan-Mathijs Schoffelen - 2011-10-03 13:31:43 +0200

OK, let's discuss in the FT-meeting.


Stephen Whitmarsh - 2011-11-29 17:19:14 +0100

Either we have to discuss this still, or my memory has to be refreshed...


Robert Oostenveld - 2012-01-11 13:19:16 +0100

make FAQ "why demean before ICA"


Eelke Spaak - 2013-05-08 15:49:29 +0200

(In reply to comment #5) I don't know immediately know the answer to this. Closing this bug and making a new bug about the FAQ.


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