Oops. Sorry, I'm pacing too much. Yes, let's get this safe + right.


On Sun, Mar 31, 2013 at 8:47 PM, Eric89GXL <notifications@github.com> wrote:

> In mne/epochs.py:
>
> > @@ -324,9 +323,7 @@ def drop_picks(self, bad_picks):
> >
> >          # XXX : could maybe be factorized
> >          self.info['chs'] = [self.info['chs'][k] for k in idx]
> > -        self.info['ch_names'] = [self.info['ch_names'][k] for k in idx]
>
> Right, I think you need info['ch_names'] instead of info['ch'].
>
> Also, it would be a good idea to add a test to make sure they were
> actually dropped from all appropriate fields. It's not ideal if this sort
> of error is only detected on human code review. That's what tests are for :)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/541/files#r3597060>
> .
>