On Wed, Jan 30, 2013 at 12:15 AM, 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:
>
> Can you add a comment why this is "passed"? What is the data being stored
> in this case? I would have assumed that, without HPI recorded, data
> starting at position 3 would just be digitization locations (
> FIFF.FIFFV_POINT_EXTRA)...
>
>
I think I will drop this option, was some previous idea but 2 more points
simply won't matter if you drop about 1000 points during alignment ...


> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/385/files#r2818288>.
>
>