Skip to content

Conversation

@bparks13
Copy link
Member

Define the number of ADCs per probe now, which should allow it to work with the Neuropixels-CAR plugin.

@jsiegle @anjaldoshi I have a question about the usage of the CAR plugin; during testing, I had to open the sidebar to see all of the streams and then choose a Neuropixels stream before it would register that it had a valid Neuropixels probe in the streams. Is there a way to update the plugin so that it maybe has a dropdown menu of all valid Neuropixels streams? Since the ONIX source has several device streams that are not Neuropixels, it could be confusing/annoying for users to have to open the sidebar and blindly choose streams until they get the one they want. On a related note, the streams pop-up does not have enough width to display the full stream name, so it cuts off and users might not be able to see the difference between the AP/LFP streams for Neuropixels 1.0.

Fixes #146

- Define the number of ADCs per probe
@bparks13 bparks13 added this to the 0.3.0 milestone Oct 27, 2025
@bparks13 bparks13 self-assigned this Oct 27, 2025
@jsiegle
Copy link
Contributor

jsiegle commented Oct 28, 2025

This change looks good! I assume you're using 32 ADCs for 1.0 probes and 24 ADCs for 2.0?

Regarding the CAR, this is not a plugin for which users would typically need to change settings, so I'm not sure a dropdown menu is worth it in this instance. But not being able to easily determine which stream is selected is a more general issue that will affect many other plugins.

Some methods that are currently available to view longer stream names: If you hover your mouse over the stream name in the stream selector, it should display the complete name. Also, if you open a channel selector interface in any plugin (e.g. Neuropixels CAR or Bandpass Filter), the top of the channel selector will display the full stream name.

That said, is it worth trying to shorten the stream names for the ONIX Source? Most of the places where stream names are displayed assumes names will be ≤16 characters. So shorter names might be more convenient from a user perspective (even if they are less informative).

@bparks13
Copy link
Member Author

bparks13 commented Nov 4, 2025

@jsiegle Yes, the metadata is using 32 ADCs for 1.0 and 24 ADCs for 2.0 probes.

That all sounds good, I definitely understand wanting to decrease the stream name length where possible. I created a new issue to discuss that: #160

If this current PR looks good we can approve and merge it, then focus on the stream names in another PR

@bparks13 bparks13 merged commit 3ddbd9a into main Nov 4, 2025
3 checks passed
@bparks13 bparks13 deleted the issue-146 branch November 4, 2025 19:54
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

Successfully merging this pull request may close these issues.

Add Metadata to Device Info for compatibility with Neuropixels-CAR plugin

4 participants