On 27.03.2013, at 22:29, Alexandre Gramfort <notifications@github.com> wrote:

> In mne/epochs.py:
> 
> >                  self._current += 1
> >                  is_good = self._is_good_epoch(epoch)[0]
> > +            # If in delayed-ssp mode, read 'virgin' data after rejection decision.
> > +            if self._delayed_ssp:
> > +                epoch = self._preprocess_epoch(epoch_raw, 'reload')
> If you don't need the epoch_raw I see 2 options : don't return it (say it with a parameter of _get_epoch_from_disk)
> 

or return a generator that first returns projected, than the raw epoch, if requested ;)

> OR have epoch_raw be the same as epoch (less clear). We can then avoir this self._preprocess_epoch(epoch_raw, 'reload'). I find the "reload" confusing.
> 
how would that work? epoch and epoch_raw cannot be the same if delayed ssp is true.
(btw. i agree, this 'reload' comes from a previous version before i introduced the 'epoch buffer'.)
> sorry for being so picky...
> 
No, that's fine. This is a non-trivial change to one of core modules. Go ahead!

> —
> Reply to this email directly or view it on GitHub.