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

Bug for Spike Detector plugin: Cannot change threshold mode for electrode larger than 19 #519

Closed
yaknerv opened this issue Aug 16, 2022 · 3 comments

Comments

@yaknerv
Copy link

yaknerv commented Aug 16, 2022

I'm working with the open-ephys acquisition board these days. Two 64 channel headstages were connected to the board. I added 128 single electrodes in the Spike Detector plugin. If I select all the electrodes, the threshold mode can be adjusted normally (i.e. using uV, STD or MED). But when I tried to adjust the threshold mode for individual electrode, I found it didn't work for electrode channel larger than 19.

A related question is, whether the ADC channels can be processed as electrode channels in the Spike Detector plugin? I tried to do this because I connected a visual stimuli onset signal to one of the ADCs. I want to detect the rising edge of a TTL signal as a spike. So I want the ADC channels work in uV mode and the electrode channels work in STD mode.

Thanks a lot for building this powerful open source system.

@yaknerv
Copy link
Author

yaknerv commented Aug 16, 2022

BTW, the gui version is 0.6.0

@jsiegle
Copy link
Member

jsiegle commented Aug 16, 2022

Thanks for reporting this! It looks like the display is failing to update for channels >19, even though the underlying threshold is actually changing. For example, in the Spike Viewer you can see the detection threshold (orange line) moving when you change the threshold, and the new threshold value is saved in the settings XML file. I'm not sure what's causing this glitch yet, but we'll figure out how to fix it ASAP.

It is possible to configure electrodes to use ADC channels as input, but it sounds like it might be better to use the Crossing Detector plugin for this application. That will create a TTL event whenever a rising edge is detected.

@anjaldoshi
Copy link
Member

Fixed in v0.6.1

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

No branches or pull requests

3 participants