diff --git a/doc/changes/latest.inc b/doc/changes/latest.inc index 7f31196711d..0649a9866aa 100644 --- a/doc/changes/latest.inc +++ b/doc/changes/latest.inc @@ -37,6 +37,8 @@ Changelog Bug ~~~ +- Fix bug in :class:`~mne.preprocessing.ICA` where requesting extended infomax via ``fit_params={'extended': True}`` was overridden, by `Daniel McCloy`_. + - Fix TAL channel parsing (annotations) for EDF-D files by `Clemens Brunner`_ - Fix handling of repeated events in :class:`mne.Epochs` by `Fahimeh Mamashli`_ and `Alex Gramfort`_ diff --git a/mne/preprocessing/ica.py b/mne/preprocessing/ica.py index 1331d1c7366..ed7ba721bd9 100644 --- a/mne/preprocessing/ica.py +++ b/mne/preprocessing/ica.py @@ -357,12 +357,10 @@ def __init__(self, n_components=None, max_pca_components=None, fit_params.update({k: v for k, v in update.items() if k not in fit_params}) elif method == 'infomax': - fit_params.update({'extended': False}) - elif method == 'extended-infomax': - fit_params.update({'extended': True}) - method = 'infomax' - if 'max_iter' not in fit_params: - fit_params['max_iter'] = max_iter + # extended=True is default in underlying function, but we want + # default False here unless user specified True: + fit_params.setdefault('extended', False) + fit_params.setdefault('max_iter', max_iter) self.max_iter = max_iter self.fit_params = fit_params