spikeinterface icon indicating copy to clipboard operation
spikeinterface copied to clipboard

Propagate "load_sync_channel" args to spikeinterface.

Open samuelgarcia opened this issue 2 years ago • 5 comments

Propagate "load_sync_channel" args to spikeinterface. And also remove HAVE_NEO_10_2 because it is a mandatory dep.

samuelgarcia avatar Jul 19 '22 12:07 samuelgarcia

@MarineChap

samuelgarcia avatar Jul 19 '22 12:07 samuelgarcia

@LiesDec : here the ability to load the synch channel. I will also push some utils function to compute the synchro with linear regression

samuelgarcia avatar Jul 19 '22 13:07 samuelgarcia

Hello, I did not test it yet but this is a modif only for spikeglx right? because we are working with open-ephys now.

MarineChap avatar Jul 20 '22 07:07 MarineChap

In OpenEphys the sync channel is not hidden I think. Unless this sync channel is stored in anthoer streams. Can you check ?

samuelgarcia avatar Jul 20 '22 08:07 samuelgarcia

I don(t know the exact detail of the problem. But here is the issue where Lies was explaining the error she gets when she try to load the sync saved as a 385e channel. https://github.com/SpikeInterface/probeinterface/issues/113

MarineChap avatar Jul 20 '22 10:07 MarineChap

@alejoe91 : have a look. ready to merge on my side if pass tests

samuelgarcia avatar Aug 29 '22 15:08 samuelgarcia