The current read_spikegadgets function name suggests it can read any SpikeGadgets .rec recording, but it only handles Neuropixels probes. It filters the HardwareConfiguration entries on name == "NeuroPixels1" and hardcodes NP1000 as the probe part number. Tetrodes and other probe types that SpikeGadgets headstages can record are silently ignored. This mirrors the misnomer that PR #427 corrected for Open Ephys.
I would like to rename the function to read_spikegadgets_neuropixels and keep read_spikegadgets as a deprecation alias that emits a DeprecationWarning, matching the pattern from #427. The alias would call the renamed function directly, so existing user code keeps working with no behavior change.
Alongside the rename, I want to add a small detector function equivalent to has_neuropixels_probes for Open Ephys. It would parse the .rec XML header and return True when at least one entry under HardwareConfiguration has a Neuropixels source name (currently "NeuroPixels1", and any future Neuropixels variants the reader grows support for). Downstream tools that want to decide whether to call the reader without catching an exception would use it.
The current
read_spikegadgetsfunction name suggests it can read any SpikeGadgets.recrecording, but it only handles Neuropixels probes. It filters theHardwareConfigurationentries onname == "NeuroPixels1"and hardcodesNP1000as the probe part number. Tetrodes and other probe types that SpikeGadgets headstages can record are silently ignored. This mirrors the misnomer that PR #427 corrected for Open Ephys.I would like to rename the function to
read_spikegadgets_neuropixelsand keepread_spikegadgetsas a deprecation alias that emits aDeprecationWarning, matching the pattern from #427. The alias would call the renamed function directly, so existing user code keeps working with no behavior change.Alongside the rename, I want to add a small detector function equivalent to
has_neuropixels_probesfor Open Ephys. It would parse the.recXML header and returnTruewhen at least one entry underHardwareConfigurationhas a Neuropixels source name (currently"NeuroPixels1", and any future Neuropixels variants the reader grows support for). Downstream tools that want to decide whether to call the reader without catching an exception would use it.