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

Pause sync sequence between trials #64

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

Conversation

glopesdev
Copy link
Member

@glopesdev glopesdev commented Feb 14, 2025

To increase robustness in logging of trial onset times where trial assets are loaded dynamically we introduce a pause in the sync sequence between trial ITI periods. This is added on top of the pseudo-random sequence in such a way that there is always a pulse at the beginning of a trial, and an offset to zero at the end of trials.

Fixes #58
Fixes #62

@glopesdev glopesdev added the feature New planned feature label Feb 14, 2025
@glopesdev glopesdev requested a review from ransona February 14, 2025 13:38
@ransona
Copy link
Collaborator

ransona commented Feb 19, 2025

the photodiode is now consistently low pre 1st trial, but is still not consistent at trial onset on the first trial. For example here's how it looks on an experiment I just recorded (yellow = PD, orange = BV sync, black = an additional PD signal that should be white for 500ms then black for 500ms):
image

I think we want it to always do a low to high transition (that lasts for the full 250ms) - this is what happens on all subsequent trials that look like this:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New planned feature
Projects
None yet
2 participants