+1 something that was on my mind for some time..

On 30.01.2013, at 00:52, Eric89GXL <notifications@github.com> wrote:

> In mne/fiff/bti/tests/test_bti.py:
> 
> > +        data = _read_data(info)
> > +        shape = (info['total_chans'], info['total_slices'])
> > +        assert_true(data.shape == shape)
> > +        info['fid'].close()
> > +
> > +
> > +def test_raw():
> > +    """ Test conversion to Raw object """
> > +
> > +    for pdf, config, hs, exported in zip(pdf_fnames, config_fnames, hs_fnames,
> > +                               exported_fnames):
> > +        # rx = 2 if 'linux' in pdf else 0
> > +        assert_raises(ValueError, read_raw_bti, pdf, 'eggs')
> > +        assert_raises(ValueError, read_raw_bti, pdf, config, 'spam')
> > +        if op.exists(tmp_raw_fname):
> > +            os.remove(tmp_raw_fname)
> Why bother removing it? Won't it overwrite it if it exists? (Do we want it to, in any case?) Perhaps we should add an "overwrite" option to the save() function...
> 
> —
> Reply to this email directly or view it on GitHub.
>