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

cued expectations #38

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

cued expectations #38

wants to merge 7 commits into from

Conversation

bimac
Copy link
Contributor

@bimac bimac commented Feb 26, 2025

We (the mesoscope wg) need your help with a task modification. For the project cortexlab_cuedexpectations (which uses the samuel_cuedBiasedChoiceWorld task), we would like to have a random subset of trials (e.g. 25%) in which the audio cue is not played at all, but everything else remains the same.
Currently the state "play tone" in the state machine code is deterministic, i.e. it triggers a 'play_tone' event on every trial. If I understand correctly, to make this probabillistic, we would need to mofidy the state machine code so that the 'play_tone' event is triggered only if the random draw is successful (or alternatively we put the volume of the sound to 0 when the draw isn't succesful). Ideally the probability of this draw would be parametrized in the task parameters.
I am not sure how to do this in practice so your help would be super appreciated!

@bimac bimac changed the title clean-up, addition of TrialDataModel, argument parsing, ruff cued expectations Feb 26, 2025
@bimac bimac self-assigned this Feb 26, 2025
@bimac bimac requested review from oliche and k1o0 March 6, 2025 13:07
@bimac bimac marked this pull request as ready for review March 6, 2025 13:08
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