also see my reviews. the nitpickers united

On 30.01.2013, at 01:42, Eric89GXL <notifications@github.com> wrote:

> In mne/fiff/bti/raw.py:
> 
> > +        info['dev_ctf_t']['trans'] = dev_ctf_t
> > +        info['ctf_head_t'] = dict()
> > +        info['ctf_head_t']['from'] = FIFF.FIFFV_MNE_COORD_CTF_HEAD
> > +        info['ctf_head_t']['to'] = FIFF.FIFFV_COORD_HEAD
> > +        info['ctf_head_t']['trans'] = ctf_head_t
> > +        logger.info('Done.')
> > +
> > +        if False:  # reminds us to support this as we go
> > +            # include digital weights from reference channel
> > +            comps = info['comps'] = list()
> > +            weights = bti_info['weights']
> > +            by_name = lambda x: x[1]
> > +            chn = dict(ch_mapping)
> > +            columns = [chn[k] for k in weights['dsp_ch_names']]
> > +            rows = [chn[k] for k in weights['ch_names']]
> > +            col_order, col_names = zip(*sorted(enumerate(columns), key=by_name))
> I had to beat @agramfort to it :)
> 
> —
> Reply to this email directly or view it on GitHub.
>