kastman/psychopy — comment on commit f203ee6929daefa2b999d8af73f419c55a6b358c
File:  line 0
By jeremygray:

On Wed, Feb 22, 2012 at 8:29 AM, Erik Kastman <
reply@reply.github.com
> wrote:

> I've been using that demo to check both the emulator and our scanner, and
> know that does work with my patch in place. I'm hoping to get a real test
> that will fail or pass as part of the suite though.
>

great!

> Were you saying before that the emulator was being used in the test suite
> in a stroop demo? If its ok with you I might put it in the "hardware"
> tests, since that's where the class lib lives and where I'd expect the
> tests to be.
>

no, I think the current test should stay in testTheApp. its a test of the
Builder (going from a psyexp file, compile to python, & execute). it uses
the ResponseEmulator class in order to do the subject responding part, so
it does provide a test of ResponseEmulator just for relying on it. but Sync
and ResponseEmulator should probably have their own proper tests--SyncEm is
untested, and if ResponseEm fails, its unlikely to give a useful output
info in testTheApp

so I definitely like your idea of having a test for SyncEmulator being
within hardware tests.

> On Feb 22, 2012, at 8:23 AM, "Jeremy R. Gray"
> wrote:
>
> > running the demo gives some performance check, but its not part of the
> official test suite. if you accept the defaults, you should get output like
> this (coder output window):
> >
> > ##### Running:
> /Users/jgray/code/psychopy/psychopy/demos/coder/experiment
> control/fMRI_launchScan.py #####
> > vol onset key
> > 0 0.000 5 start of scanning run, vol 0
> > 0 0.123 a
> > 1 2.000 5
> > 1 2.456 b
> > 2 4.000 5
> > 2 4.789 c
> > 3 6.000 5
> > 4 8.000 5
> > end of scan (vol 0..4 = 5 of 5). duration = 10.000
> > For the test, there should be 5 trials (vol 0..4, key 5), with three
> simulated subject responses (a, b, c)
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/kastman/psychopy/commit/f203ee6929daefa2b999d8af73f419c55a6b358c#commitcomment-995850
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kastman/psychopy/commit/f203ee6929daefa2b999d8af73f419c55a6b358c#commitcomment-995871
>