That makes sense. Might as well leave them.
On Feb 16, 2013 5:28 PM, "Denis A. Engemann" <notifications@github.com>
wrote:

> In mne/fiff/bti/raw.py:
>
> > +
> > +# Authors: Denis A. Engemann  <d.engemann@fz-juelich.de>
> > +#          Martin Luessi <mluessi@nmr.mgh.harvard.edu>
> > +#          Alexandre Gramfort <gramfort@nmr.mgh.harvard.edu>
> > +#          Matti Hamalainen <msh@nmr.mgh.harvard.edu>
> > +#          Yuval Harpaz <yuvharpaz@gmail.com>
> > +#
> > +#          simplified bsd-3 license
> > +
> > +from .. import Raw
> > +from .. import FIFF
> > +from .  constants import BTI
> > +from . read import read_int32, read_int16, read_str, read_float, read_double,\
> > +                   read_transform, read_char, read_int64, read_uint16,\
> > +                   read_uint32, read_double_matrix, read_float_matrix,\
> > +                   read_int16_matrix
>
> Eric, I can see your point, but actually I'm thinking that just for
> wrapping np.fromstring this isn't worht it. I woud basically change the
> lines saying _unpack_simple and drop this privat function. That's not a
> great advantage. Also I tried it and it won't work in each case. Changing
> the read_XXX functions is not good either, I find them quite supporting for
> readability.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/385/files#r3040303>.
>
>