Skip to content

ENH: Add function to get parameters from stimuli, which you can use to recreate it#6801

Merged
TEParsons merged 6 commits intopsychopy:devfrom
TEParsons:dev-enh-stim-serialization
Sep 20, 2024
Merged

ENH: Add function to get parameters from stimuli, which you can use to recreate it#6801
TEParsons merged 6 commits intopsychopy:devfrom
TEParsons:dev-enh-stim-serialization

Conversation

@TEParsons
Copy link
Copy Markdown
Contributor

The main use of this is ioHub - because it runs in a separate process, we can't pass object handles directly, so have to recreate any stimuli. Making all stimuli serializable means we can pass a JSON string between processes and use it to recreate the stimulus on the other end, meaning any stimulus is then able to use used in calibration/validation, not just TargetStim.

@TEParsons TEParsons changed the title ENH: Add function to get "serialized" stimuli - a dict of values from which it can be recreated ENH: Add function to get parameters from stimuli, which you can use to recreate it Aug 20, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 20, 2024

Codecov Report

Attention: Patch coverage is 93.50649% with 5 lines in your changes missing coverage. Please review.

Project coverage is 49.73%. Comparing base (ca82e91) to head (0516463).
Report is 109 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #6801      +/-   ##
==========================================
+ Coverage   49.62%   49.73%   +0.10%     
==========================================
  Files         332      332              
  Lines       61218    61294      +76     
==========================================
+ Hits        30381    30482     +101     
+ Misses      30837    30812      -25     
Components Coverage Δ
app ∅ <ø> (∅)
boilerplate ∅ <ø> (∅)
library ∅ <ø> (∅)
vm-safe library ∅ <ø> (∅)

@TEParsons TEParsons merged commit b8e169f into psychopy:dev Sep 20, 2024
@TEParsons TEParsons deleted the dev-enh-stim-serialization branch October 4, 2024 11:13
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.

1 participant