then let's at least try to catch some corner cases. 
antways. will not matter too much as this PR intends to make the ICA class much more expressive.

On 07.02.2013, at 19:08, Alexandre Gramfort <notifications@github.com> wrote:

> In mne/preprocessing/ica.py:
> 
> > -
> > -    logger.info('Ready.')
> > -
> > -    return ica
> > +    try:
> > +        ica = _detect_artifacts(ica=ica, raw=raw, start_find=start_find,
> > +                stop_find=stop_find, ecg_channel=ecg_channel,
> > +                ecg_score_func=ecg_score_func, ecg_criterion=ecg_criterion,
> > +                eog_channel=eog_channel, eog_score_func=eog_score_func,
> > +                eog_criterion=ecg_criterion, skew_criterion=skew_criterion,
> > +                kurt_criterion=kurt_criterion, var_criterion=var_criterion,
> > +                add_nodes=add_nodes)
> > +    except:
> > +        logger.info('Detection failed. Returning ica without indices marked'
> > +                    ' for exclusion. Please continue using the '
> > +                    'ica.detect_artifacts method')
> I understand this argument when you debug but for production code I don't get it. it should not happen or fail gracefully.
> 
> —
> Reply to this email directly or view it on GitHub.
>