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

set the MAIN SYNC parameter from iblrig, in a safe manner #640

Open
oliche opened this issue Mar 25, 2024 · 0 comments
Open

set the MAIN SYNC parameter from iblrig, in a safe manner #640

oliche opened this issue Mar 25, 2024 · 0 comments
Assignees

Comments

@oliche
Copy link
Member

oliche commented Mar 25, 2024

The Slack thread is there: https://int-brain-lab.slack.com/archives/C012URA26TE/p1711100988083029

Thanks Mayo, cross-posting this to the HW channel.
Right now they would have to manually edit the hardware_settings. I did not show them while in Seattle on purpose (sorry).
My fear was that getting MAIN_SYNC wrong for the behaviour has a modest impact, while getting MAIN_SYNC wrong for ephys has a large impact, and I was afraid that the back and forth between Kim's experiments and Matt training would lead to ephys problems.
Right now there is no interface, but we could add a field in the GUI, with a safeguard, options could be either or both of:
if there is ephys in the rig name and MAIN_SYNC is True, throw a confirmation dialog asking to confirm this is a training session
if the ephys protocol is chosen and MAIN_SYNC is True, throw an error dialog preventing the user to go forward

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

No branches or pull requests

2 participants