SpikeInterface / spikeinterface

A Python-based module for creating flexible and robust spike sorting pipelines.
https://spikeinterface.readthedocs.io
MIT License
517 stars 184 forks source link

Problem when reading neuralynx header from Cheetah 5.6.2 #2007

Closed JohnStout closed 4 months ago

JohnStout commented 1 year ago

Using the spikeinterface.extractors.read_neuralynx does not seem to work on headers created using Cheetah software 5.6.2. Works fine on header from Cheetah 6.4.1.

Here is a link to google drive with two CSC.ncs files (one that works with read_neuralynx and one that doesn't work for me) https://drive.google.com/drive/folders/1RRVdPkOuQY946adaJOEH6Sp0w0XwmISh?usp=share_link

Attached is an image of the error.

Screenshot 2023-09-18 at 3 24 17 PM
zm711 commented 1 year ago

@JohnStout

Looks like there is an open PR at https://github.com/NeuralEnsemble/python-neo/pull/1257 (and associated issue https://github.com/NeuralEnsemble/python-neo/issues/1202) trying to deal with Cheetah 5.6.x. One of the problems is that they need test files that could be included in the public test repository to ensure the fixes work at the Neo level. If you'd be willing to share your failing file as part of the test suite it may facilitate getting that PR merged, which then could be propagated to SpikeInterface.

JohnStout commented 1 year ago

Hey Zach,

Did you receive those google drive files? Two example files, one with newer version of cheetah. One with older. They're just folders with CSC files in them.

Do you happen to know of any short term solutions?

Thanks! John

On Mon, Sep 18, 2023 at 5:12 PM Zach McKenzie @.***> wrote:

@JohnStout https://github.com/JohnStout

Looks like there is an open PR at NeuralEnsemble/python-neo#1257 https://github.com/NeuralEnsemble/python-neo/pull/1257 (and associated issue NeuralEnsemble/python-neo#1202 https://github.com/NeuralEnsemble/python-neo/issues/1202) trying to deal with Cheetah 5.6.x. One of the problems is that they need test files that could be included in the public test repository to ensure the fixes work at the Neo level. If you'd be willing to share your failing file as part of the test suite it may facilitate getting that PR merged, which then could be propagated to SpikeInterface.

— Reply to this email directly, view it on GitHub https://github.com/SpikeInterface/spikeinterface/issues/2007#issuecomment-1724428944, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK7U57VP5X64EDRQQ5YRBW3X3C2LBANCNFSM6AAAAAA45GOEYQ . You are receiving this because you were mentioned.Message ID: @.***>

zm711 commented 1 year ago

https://spikeinterface.readthedocs.io/en/latest/installation.html

Goes through how to install spikeinterface and neo from source. But in your case you would need to use the branch of neo with the fixes rather than the main neo. Are you comfortable enough with pip and git to do that yourself?

JohnStout commented 1 year ago

Yup!

On Tue, Sep 19, 2023 at 11:28 AM Zach McKenzie @.***> wrote:

https://spikeinterface.readthedocs.io/en/latest/installation.html

Goes through how to install spikeinterface and neo from source. But in your case you would need to use the branch of neo with the fixes rather than the main neo. Are you comfortable enough with pip and git to do that yourself?

— Reply to this email directly, view it on GitHub https://github.com/SpikeInterface/spikeinterface/issues/2007#issuecomment-1725893061, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK7U57SEXWDTWJ42KLQQ5DDX3G22VANCNFSM6AAAAAA45GOEYQ . You are receiving this because you were mentioned.Message ID: @.***>

zm711 commented 7 months ago

@JohnStout Neo has updated the neuralynx .rawio. If you still need it we'd love to hear if the updates have worked for you.

zm711 commented 4 months ago

I'll close this for now. There has been a lot of work at the Neo level. If someone else runs into this in the future feel free to open an issue on Neo.

JohnStout commented 4 months ago

Sorry for not responding! Thank you! Haven’t used this interface in a bit to know!

On Wed, Jun 19, 2024 at 12:33 PM Zach McKenzie @.***> wrote:

I'll close this for now. There has been a lot of work at the Neo level. If someone else runs into this in the future feel free to open an issue on Neo.

— Reply to this email directly, view it on GitHub https://github.com/SpikeInterface/spikeinterface/issues/2007#issuecomment-2179096725, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK7U57ULBQPTUBGXBKTCKNTZIGXERAVCNFSM6AAAAAA45GOEYSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZZGA4TMNZSGU . You are receiving this because you were mentioned.Message ID: @.***>