-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adds Interfaces for SpikeGadgets and Video #2
Conversation
@@ -3,7 +3,7 @@ channels: | |||
- conda-forge | |||
- defaults | |||
dependencies: | |||
- python>=3.9 | |||
- python>=3.9, <3.13 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neo doesn't support 3.13 yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Just a small note, I can see two devices in the stub nwbfile, one is called DeviceEcephys and the other one ProbeNameTBD. I guess they refer to the same device but the first is added by default, you might want to remove that. Correct me if I am wrong.
Yeah, that's right. I will need to remove the default device when I get the probe info from Jacob. |
Adds Neuroconv's VideoInterface, and a semi-custom
Olson2024SpikeGadgetsRecordingInterface(SpikeGadgetsRecordingInterface)
to appropriately handle all the metadata.This PR depends on this fix for neo.