Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stimuli and delays will be longer to help saveToCSV save triggers pro… #123

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kylemath
Copy link
Owner

…perly

@kuziekj - Next trial will not start until stimuli have been presented for 500 ms. Also increased the minimum delay between trials to 1500 ms. Basically created a flag (waitForStim) that indicates if the stimuli should be removed on the current frame or not. This seems to fix the issues we were having where saveToCSV was getting bogged down and not saving certain stimulus triggers.

@kylemath
Copy link
Owner Author

@kuziekj - I made some changes o your code to remove some repetition, but it does the same thing you implemented, keeping the stim on the screen for a while (and the marker up), can you test this out to make sure it is still working

also made an issue #124

@korymath
Copy link
Collaborator

What is the status on this?
Ready for review?

@kylemath
Copy link
Owner Author

no no, working on it laster in the semester once we get to that module,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants