nisl/tutorial — pull request #49 by pgervais (2013-04-23 13:25:45+00:00)
Title: Fixed a bug in signals.clean()

The "detrend" boolean argument in clean() is an easy way to add a trend as another confound to remove. Thus, it must be exactly equivalent to adding a trend in the "confounds" array. This was not the case before this commit.

The fix was easy: it was just a matter of detrending the confounds as well as the signals.

A similar concern could exist with filtering: if a low-pass or high-pass filter is specified, the output result must not contain unfiltered frequencies, even if some confounds do contain those frequencies. This is not the case with the present implementation, because filtering is performed after everything else.