Conversation
- Define the number of ADCs per probe
|
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). |
|
@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 |
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