yeah, fine with me!

On 19.07.2013, at 20:15, Roman Goj <notifications@github.com> wrote:

> In mne/time_frequency/tests/test_csd.py:
> 
> > +event_id, tmin, tmax = 1, -0.2, 0.5
> > +events = mne.read_events(event_fname)
> > +epochs = mne.Epochs(raw, events, event_id, tmin, tmax, proj=True,
> > +                    picks=picks, baseline=(None, 0), preload=True,
> > +                    reject=dict(grad=4000e-13, mag=4e-12))
> > +
> > +
> > +def test_compute_csd():
> > +    """Test computing cross-spectral density from epochs
> > +    """
> > +    # Check that wrong parameters are recognized
> > +    assert_raises(ValueError, compute_csd, epochs, mode='notamode')
> > +    assert_raises(ValueError, compute_csd, epochs, fmin=20, fmax=10)
> > +    assert_raises(ValueError, compute_csd, epochs, tmin=0.15, tmax=0.1)
> > +    assert_raises(ValueError, compute_csd, epochs, tmin=0, tmax=10)
> > +    assert_raises(ValueError, compute_csd, epochs, tmin=10, tmax=11)
> Hmm, my preference would be not to have a loop in such a case, i.e. for tests. To me a loop would make it harder to see what exactly failed in a test, wdyt?
> 
> —
> Reply to this email directly or view it on GitHub.