-
Notifications
You must be signed in to change notification settings - Fork 34
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
Matrix Copy Phrase Task #261
Conversation
…ly writing to properties
…ant creation of TextStim elements which could affect screen draw time; use configured parameters for pause length; added ability to distinguish between inquiries that display the target and those that don't; updated matrix tasks to utilize the new functionality
…eters needed for matrix display
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I finished testing this locally last week, and it looks good to me! I like the little cleanups you did. After a successful human-in-the-loop test, you can merge this. I'll do a group slack message to arrange it.
… ignored fixture color for copy phrase, which does not use the color parameters for stim
Matrix calibration refinements
We ran the Matrix Timing Task using the photodiode and analyzed the offsets:
Using the recommended offset:
|
Overview
Added a Matrix Copy Phrase Task.
Ticket
https://www.pivotaltracker.com/story/show/180699453
Contributions
Test
matrix_copy_phrase.mov