sure :-) I see it was the right moment to start a discussion. Too many hours on my own at this one ;-)

On 30.01.2013, at 00:31, Eric89GXL <notifications@github.com> wrote:

> In mne/fiff/bti/raw.py:
> 
> > +    idx_idents = range(1, 4) + range(1, (len(idx_points) + 1) - 3)
> > +    dig = []
> > +    for idx in xrange(all_points.shape[0]):
> > +        point_info = dict(zip(FIFF_INFO_DIG_FIELDS, FIFF_INFO_DIG_DEFAULTS))
> > +        point_info['r'] = all_points[idx]
> > +        if idx < 3:
> > +            point_info['kind'] = FIFF.FIFFV_POINT_CARDINAL
> > +            point_info['ident'] = idx_idents[idx]
> > +        if 2 < idx < len(idx_points) and use_hpi:
> > +            point_info['kind'] = FIFF.FIFFV_POINT_HPI
> > +            point_info['ident'] = idx_idents[idx]
> > +        elif idx > 4:
> > +            point_info['kind'] = FIFF.FIFFV_POINT_EXTRA
> > +            point_info['ident'] = (idx + 1) - len(idx_idents)
> > +
> > +        if 2 < idx < len(idx_points) and not use_hpi:
> Ahh, I see. I was worried maybe it would be the first two EEG locations in an M/EEG recording or something :)
> 
> It would be good to add a comment along those lines so that those of us unfamiliar with the system can understand the motivation simply from looking at the code.
> 
> —
> Reply to this email directly or view it on GitHub.
>