Back to the main page.

Bug 228 - preprocessing option cfg.blc should be renamed

Status CLOSED FIXED
Reported 2010-11-19 11:55:00 +0100
Modified 2011-01-06 09:17:03 +0100
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P1 normal
Assigned to: Jan-Mathijs Schoffelen
URL:
Tags:
Depends on:
Blocks:
See also:

Robert Oostenveld - 2010-11-19 11:55:29 +0100

For many people it is not clear that blc refers to BaseLineCorrection. We should therefore rename this option. Better is (note the consistency) cfg.demean = yes/no cfg.detrend = yes/no cfg.baselinewindow = [beg end] Also the other functions that do something with baseline "correction" (e.g. TFRs relative to baseline) should be cleaned up. TODO: discuss in T meeting


Robert Oostenveld - 2010-12-08 14:42:56 +0100

this should use ft_checkconfig


Jan-Mathijs Schoffelen - 2010-12-15 09:47:42 +0100

renamed blc into demean and blcwindow into baselinewindow throughout the code (fieldtrip and fieldtrip/private). Functions using cfg.blc/cfg.blcwindow now call ft_checkconfig for backward compatibility. There is an additional check in preproc.