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

“Apply to all” appears to, but does not actually switch channels being recorded in the GUI and the settings.xml. #592

Open
egmcbride opened this issue Dec 4, 2023 · 8 comments

Comments

@egmcbride
Copy link

Using OpenEphys 0.6.6 and Neuropixels-PXI, we set the middle bank of channels (384-768) to be recorded by loading a saved .imro file for a single probe, and verified the signals on that probe are noticeably different in the way we expect, i.e. a clear in- to out-of-brain transition for a deep insertion.

Then after pressing “apply to all”, the new channel banks appear in the GUI to be selected on the other probes, but on closer inspection, the signals have not changed, and they are still recording from the original channels (the lowest 384). Then, when loading the .imro file for each individual probe, the signals do change in the way expected. When recording after pressing apply to all, the settings.xml file reflects what is shown in the GUI, the incorrect channels.

It would be very easy to miss this if you aren't looking closely at the signals or inspecting the raw data.

@jsiegle
Copy link
Member

jsiegle commented Dec 6, 2023

Thanks for posting this! It's been tricky to debug because we haven't been able to replicate this behavior on any of our dev machines. Which rig were you seeing this on? Can we run some tests on it when it's free?

@bjhardcastle
Copy link
Contributor

bjhardcastle commented Dec 6, 2023

A while back we found that remote configuration of the channels on this rig (NP.3) also wasn't working, where it works fine on other rigs with the same version and same HTTP request.

One obvious difference on the rig is a NI-DAQmx source in the signal chain, and a Merger with the Neuropix-PXI source - I wonder whether that could be the cause?

Edit: removing NI-DAQmx source and Merger had no effect.

@egmcbride
Copy link
Author

Also, NP3 will be free this Friday afternoon (12/8) if you want to run some tests then.

@jsiegle
Copy link
Member

jsiegle commented Dec 6, 2023

Thanks! I'm busy on Friday, but let's try to find some time next week

@jsiegle
Copy link
Member

jsiegle commented Dec 12, 2023

We were able to replicate this by starting acquisition before all of the settings were done updating. This was only using 2 probes, but for 6 probes the likelihood that this would happen accidentally is much higher, since each probe takes ~2 seconds to apply its settings.

We are going to address this in two ways: by increasing the waiting period for the settings thread to exit after acquisition starts, and by adding some logging outputs that make it clear when the thread has exited early.

This will hopefully ensure that the settings get applied correctly across all probes. But just to be on the safe side, for future recordings I'd also recommend doing the following:

  • Wait at least 20 seconds for the settings to finish updating after pressing the "apply to all" button. If the port icons in the Neuropix-PXI editor turn green, then this indicates enough time has elapsed. This also applies to cases where the electrode configuration is being updated remotely.
  • Visually confirm that the signals have changed in the LFP Viewer.

@jsiegle
Copy link
Member

jsiegle commented Dec 21, 2023

In the latest version of the Neuropixels plugin (available now), pressing the "Apply to all" button displays a progress bar until all of probe settings have been updated. It also avoids re-applying the settings for the currently active probe, which should make the update happen a little bit faster.

In addition, you can now select preset electrode configurations from a drop-down menu, so it's much easier to switch between different channel banks.

@bjhardcastle
Copy link
Contributor

Thanks a lot Josh, the dropdown is great!

@bjhardcastle
Copy link
Contributor

We'll update on whether the channels actually switch for us using this method.

Apparently waiting longer didn't work.

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