On Wed, Mar 27, 2013 at 9:22 AM, Alexandre Gramfort <
notifications@github.com> wrote:

> In mne/epochs.py:
>
> > @@ -386,26 +415,12 @@ def _get_epoch_from_disk(self, idx, verbose=None):
> >          if start < 0:
> >              return None
> >          epoch, _ = self.raw[self.picks, start:stop]
> > -
> > -        if self.proj and self._projector is not None:
> > -            logger.info("SSP projectors applied...")
> > +        if self.reject is not None and self.proj == False:
> > +            self._epoch_tmp = epoch.copy()
>
> yes keep _preprocess_epoch but if you can avoid the _epoch_tmp private
> attribute it would be nice. You could also avoid reading the epoch twice by
> just keeping both versions in memory. I find it much cleaner.
>
> That's what the ._tmp_epoch was for, I did not read it twice ;-)
But I agree, your proposal will make it more readable and cleaner.


> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/487/files#r3546458>
> .
>