VEP Display #237
Merged
VEP Display #237
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lawhead
approved these changes
Aug 26, 2022
| vep = VEPDisplay(win, experiment_clock, stimuli, task_display, info) | ||
| timing = [] | ||
| t = 2 | ||
| for i in range(len(task_text)): |
There was a problem hiding this comment.
Alternatively, this could be:
for (txt, color) in zip(task_text, task_color):
vep.update_task(txt, color[0])
|
|
||
|
|
||
| def create_vep_codes(length=32, count=4) -> List[List[int]]: | ||
| """Create a list of VEP codes.""" |
There was a problem hiding this comment.
It would be good to document what length and count are used for so it's clear when used elsewhere.
| np.random.seed(1) | ||
| for _ in range(count): | ||
| res.append(np.random.randint(2, size=length)) | ||
| return res |
There was a problem hiding this comment.
This could be reduced to a list comprehension:
return [np.random.randint(2, size=length) for _ in range(count)]
|
|
||
| This assumes that vep_type is 4 using a 2x2 grid. Additional configurations can be added in the future. | ||
| """ | ||
| # generate the inquiry stimuli |
There was a problem hiding this comment.
Is the idea that we will have a fixed number of configurations, or did you have thoughts on making this more general and be able to accommodate any possible grid?
There was a problem hiding this comment.
I really only considered having a few fixed layouts with different size flickering VEP boxes. I'd be open to something more configurable within a fixed unit type (height or norm). We could try looping in code generated via psychopy to give more options. When coding this, I realized we aren't really equipped across the board to support interface building for non-coding folks (only configuration).
If a user wanted something more custom, I would recommend they build it in the psychopy experiment builder GUI and import bcipy to get the VEP flickering and/or acquisition.
Merged
tab-cmd
added a commit
that referenced
this pull request
Oct 3, 2022
* add timer to prevent double clicking (#218) * #181963356 ; added better alerting to inform users when fake data mode is on * Added changelog entry * Refactoring to reduce duplication * #182040775 ; option to summarize session at the end of a copy phrase task * Update README.md High level GPT2 documentation added, word level prediction described, contact information added * Matrix Display Fixes (#221) * fixes bug with last stimuli lagging in matrix calibration, added entire grid flash before each inquiry * Address PR comment + cleanup Co-authored-by: Tab Memmott <[email protected]> * Refactored session helper to generate spreadsheets without first creating a sqlite database * ERP plotting improvements and MNE data support (#220) * #182005453 ; added parameter for signal model path; updated initialization code to use this for typing tasks * #182406904 ; Fix bug when generating a spreadsheet from active session data * [Bug fix] Two presses needed to start copy phrase (#225) * Documentation and formatting * #182200897 ; check for internet connection and confirm before proceeding with the session * Added logging for GPU information, if available * Whitespace fix * Added changes * Move session validation to validate module; update internet connectivity test to work with VPN * #182106570 ; added functionality for configuration channel specifications for a given device. * Added a test mock to prevent a GUI interaction during testing * Fix test issue by switching to a different acquisition client for mocking * use a better default timeout (#228) * #182776686 ; prevent widgets scrolling on parameters form * Jitter (#233) Jitter for calibration + parameter. copy phrase ip cleanup stimuli tests * User ID reset bug and cleanup (#235) User id reset fix * Add support for Tobii Nano (#234) * Add support for Tobii Nano * address PR feedback * update changelog * added ssvep to cvep method (#232) Co-authored-by: Aida Fakhry <[email protected]> Co-authored-by: Tab Memmott <[email protected]> * #183037159 ; alert when using battery power * Better documentation on return values and potential impacts * VEP Display (#237) * upgrade psychopy (#239) * ERP viz demo fix (#240) * #175193924 infrastructure for changed parameters ; #183134229 Fixed issue with params_form rounding some float values ; #183133776 allow static offset to be input with more precision * Remove unneeded parameters (#241) * signal model loaded log (#244) better signal model logging * Devices config update && BciPy running outside of root (#242) * Refactor param changes GUI to use a scroll area for displaying a large number of changes * Refinements; parameters changes start out collapsed if empty; minor refactorings * #182408060 ; Ensure that spelled text is provided to the language model when using fake data * Export the device used as a useable config file (#247) * Convert: add BDF support and fix static offset bug (#246) * #182031676 ; remove TCP-based acquisition code * Added PR number to changelog * Added function type information; better string representations of channel specs and device specs; removed unused code in tests * Updated max_buffer_len parameter name for lsl_client * fix rebase issue in parameters module * Bug fixes: jitter, offline analysis, task complete alerting (#250) * Matrix Bug: fix for scp flashing (#252) * Matrix documentation (#253) Co-authored-by: lawhead <[email protected]> Co-authored-by: Shijia Liu <[email protected]> Co-authored-by: jcgangemi1 <[email protected]> Co-authored-by: AidaFakhry <[email protected]> Co-authored-by: Aida Fakhry <[email protected]> Co-authored-by: Basak Celik <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Added a VEP Display component.
Note: while I've made a method for animation and a variable referencing vep_type, these need more direction from the clinical and engineering teams (4x4 vs 6x6; how it should animate). I want to preserve all of the spike work we've done thus far without committing to fancy animations or code generation at this time. Please add any suggestions that will make future changes easier.
Ticket
https://www.pivotaltracker.com/story/show/182457440
Contributions
display/paradigm/vep.py:VEPDisplay: main display component needed to operate in any task.VEPBox: Bounding box for every single flickering box.create_vep_codes: method to create a random code for displaying the vep flickering.display/paradigm/rsvp.py: Add typing / linting.bcipy/display.py: addedVEPStimulusPropertiesto better encapsulate the stimuli needed for VEP operationdisplay/demo/vep/demo_calibration_vep.py: a script to demonstrate how to set up and use the VEPDisplay in a task type setting.Test
display/demo/vep/demo_calibration_vep.pyRan and made sure everything hit AC. No other parts of the code were touched.IMG_9307.MOV
Documentation
Changelog