-
Notifications
You must be signed in to change notification settings - Fork 1
Automatically save Probe Interface files when recording starts #144
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
base: main
Are you sure you want to change the base?
Conversation
- Introduced NeuropixelsProbeMetadata class to encapsulate probe metadata retrieval and management. - Updated Neuropixels1e, Neuropixels1f, and Neuropixels2e classes to utilize the new metadata class for serial number, part number, and flex version retrieval. - Enhanced UI components to display probe part number and flex information. - Improved error handling and logging for metadata reading processes.
Attached here is a zip file with the folder structure shown above. The ephys data has been removed to keep the file size small. |
Attached here are three more electrode configurations from a 1.0 probe to test the loading scripts. The specific configurations used are listed below, where the electrode indices are zero-indexed and are the global index of the electrode:
|
- Standardize how stream names are created across the code base for 2.0 probes - Standardized the stream name itself for 2.0 probes to include the device name, similar to other devices
Attached here are additional recordings using two Neuropixels 2.0 probes in one headstage, with the given electrode configurations for each experiment
|
Question about this dataset, looking at |
@anjaldoshi Can you provide any insight into this? The probe part number is supposed to be read directly from hardware, so if this is incorrect I will need to put a fix in. |
@chrishalcrow We figured it out, the I have created a new issue on the plugin repo to add support for single-shank probes so that we do not report incorrect electrode indices in the future. Sorry about the confusion with that! |
That's great - no worries about the confusion, glad we caught the bug. |
Hi @bparks13 I added the first set of settings as tests and cleaned up the probeinterface reader on our end. Can you check if everything looks good? |
Waiting for this to be fixed to add some NP2 and multiprobe test files :) |
Summary
This PR introduces automatic saving of Probe Interface files when recording starts. The changes are aimed at improving compatibility with SpikeInterface and simplifying downstream file processing by providing JSON metadata directly.
Key Changes
Motivation
These updates are primarily intended to support SpikeInterface compatibility, reducing friction during data analysis by allowing metadata to be loaded directly from a JSON file.
Folder Structure
When a recording is started, the data directory can contain two top-level folders:
experiment1
,experiment2
,experiment3
, etc.) whenever there are streams that need to save a Probe Interface file<stream_name>.json
file, where the filename matches the associated stream. More than one JSON file can exist in each folder.experiment1
,experiment2
,experiment3
, etc.).settings.xml
,settings_2.xml
,settings_3.xml
, etc.This layout ensures that probe metadata is saved alongside recording data, with stream-specific JSON files simplifying downstream processing and analysis. For a visual representation of the possible folder structure, see below:
It is important to note two things:
Fixes #135