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

SpikeGLXNIDQInterface interface metadata #1154

Open
h-mayorquin opened this issue Dec 11, 2024 · 0 comments
Open

SpikeGLXNIDQInterface interface metadata #1154

h-mayorquin opened this issue Dec 11, 2024 · 0 comments

Comments

@h-mayorquin
Copy link
Collaborator

Currently, we are working on adding digital channels to SpikeGLXNIDQInterface and changing the data type from ElectricalSeries to TimeSeries (see #1152). The goal of the SpikeGLXConverterPipe is to automatically include as much information and metadata as possible, which includes handling NIDQ data.

NIDQ devices can acquire signals of very different natures. These can represent anything from temperature and motion to MEG or other non-neural physiological markers. Unfortunately, SpikeGLX metadata does not include channel-level descriptions (reference), so we need to provide a way for users to specify this metadata and store it appropriately.

Digital channels
Each digital channel is written as its own ndx-events object, making it straightforward to annotate these channels individually. However, analog channels present a different issue.

Analog channels
Currently, all analog signals are written into a single TimeSeries. The problem is that TimeSeries does not support per-channel descriptions—only a single, general description. One possible workaround is to include a table in the TimeSeries description with details for each channel. Another option is to create one TimeSeries per channel, writing the channel’s description directly into that TimeSeries.

Missing features

  • If we choose the one-TimeSeries-per-channel approach for analog signals, it would help to link all these TimeSeries objects to the same device.
  • If we keep a single TimeSeries for all analog signals, it would be ideal to have per-channel descriptions rather than a single description for the entire dataset.
    Neither linking multiple TimeSeries objects to a single device nor providing per-channel descriptions is currently supported by TimeSeries. I will open issues in pynwb to discuss these options.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant