NeuralEnsemble / python-neo

Neo is a package for representing electrophysiology data in Python, together with support for reading a wide range of neurophysiology file formats
http://neo.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
307 stars 240 forks source link

Fix issue #1432: loading Neuropixel data from SpikeGadgets session. #1433

Closed gkBCCN closed 2 months ago

gkBCCN commented 2 months ago

Updates spikegadgetsrawio.py:_parse_header() to check devices for proper dictionary keys before attempting to access them.

alejoe91 commented 2 months ago

Thanks @gkBCCN

alejoe91 commented 2 months ago

@gkBCCN could you provide a very very short dataset for testing?

gkBCCN commented 2 months ago

Hi @alejoe91 I uploaded a very short (~3s) recording .rec file from 2 Neuropixels 1.0 probes: here.

alejoe91 commented 2 months ago

That's a bit too large. Can you make it as small as possible? :) We upload the test data here: https://gin.g-node.org/NeuralEnsemble/ephy_testing_data

Each entry should be ideally less than 30MB..so that would be a very very short for NP probes. If you have access to the system, oculd you give that a try?

alejoe91 commented 2 months ago

@gkBCCN we added the test file to the ephy_testing_data

Can you add it to the NEO tests? You just need to add the dataset here

gkBCCN commented 2 months ago

Thank you for adding the .rec to the ephys testing data set for me. Yes, I will update the NEO tests.

alejoe91 commented 2 months ago

Fixes #1432