Fix failing tests (#4560)

Co-authored-by: Chris Halcrow <57948917+chrishalcrow@users.noreply.github.com>

Author: alejoe91

doc/releases/0.104.1.rst

@@ -0,0 +1,26 @@
+.. _release0.104.1:
+
+SpikeInterface 0.104.1 release notes
+------------------------------------
+
+April 7th 2025
+
+Minor release with bug fixes
+
+
+extractors:
+
+* Align CompressedBinaryIBLExtractor with updated get_neuropixels_sample_shifts (#4477)
+
+
+postprocessing:
+
+* Make get_data for valid unit periods use unit ids, not indices ( #4468)
+* Remove tmp_data_to_save when loading old analyzers (#4480)
+* Handle peak_sign for quality metrics in _handle_backward_compatibility_on_load (#4481)
+
+
+Contributors:
+
+* @chrishalcrow
+* @hayleybounds

doc/releases/0.104.2.rst

@@ -0,0 +1,24 @@
+.. _release0.104.2:
+
+SpikeInterface 0.104.2 release notes
+------------------------------------
+
+April 20th 2025
+
+Minor release with bug fixes
+
+core:
+
+* Fix ``WaveformExtractor`` backwards compatibility: run backwards compatibility migration before validating metrics (#4509)
+* Make version mismatch warning informative and actionable (#4505)
+
+extractors:
+
+* Open Ephys: find root folder to correctly look for timestamps (#4500)
+
+
+Contributors:
+
+* @alejoe91
+* @h-mayorquin
+* @galenlynch

doc/releases/0.104.3.rst

@@ -0,0 +1,21 @@
+.. _release0.104.3:
+
+SpikeInterface 0.104.3 release notes
+------------------------------------
+
+April 22th 2025
+
+Minor release with bug fixes
+
+preprocessing:
+
+* Fix filter kwargs (#4543)
+
+widgets:
+
+* Fix sortingview "backend" in plot_sorting_summary (#4554)
+
+
+Contributors:
+
+* @alejoe91

doc/whatisnew.rst

@@ -8,6 +8,9 @@ Release notes
 .. toctree::
   :maxdepth: 1
 
+  releases/0.104.3.rst
+  releases/0.104.2.rst
+  releases/0.104.1.rst
   releases/0.104.0.rst
   releases/0.103.2.rst
   releases/0.103.1.rst
@@ -52,6 +55,21 @@ Release notes
   releases/0.9.9.rst
   releases/0.9.1.rst
 
+Version 0.104.3
+===============
+
+* Minor release with bug fixes
+
+Version 0.104.2
+===============
+
+* Minor release with bug fixes
+
+Version 0.104.1
+===============
+
+* Minor release with bug fixes
+
 Version 0.104.0
 ===============
 

pyproject.toml

@@ -268,6 +268,10 @@ test-curation = [{include-group = "test-common"}]
 test-widgets = [
     {include-group = "test-common"},
     "sortingview>=0.12.0",
+    "numba>=0.59",  # For motion widget
+    "scikit-learn",
+    "pyvips",
+    "networkx",
 ]
 
 test-exporters = [{include-group = "test-common"}]

src/spikeinterface/extractors/neoextractors/spikeglx.py

@@ -23,9 +23,6 @@ class SpikeGLXRecordingExtractor(NeoBaseRecordingExtractor):
     ----------
     folder_path : str
         The folder path to load the recordings from.
-    load_sync_channel : bool default: False
-        Whether or not to load the last channel in the stream, which is typically used for synchronization.
-        If True, then the probe is not loaded.
     stream_id : str or None, default: None
         If there are several streams, specify the stream id you want to load.
         For example, "imec0.ap", "nidq", or "imec0.lf".
@@ -40,24 +37,21 @@ class SpikeGLXRecordingExtractor(NeoBaseRecordingExtractor):
     Examples
     --------
     >>> from spikeinterface.extractors import read_spikeglx
-    >>> recording = read_spikeglx(folder_path=r'path_to_folder_with_data', load_sync_channel=False)
-    # we can load the sync channel, but then the probe is not loaded
-    >>> recording = read_spikeglx(folder_path=r'pat_to_folder_with_data', load_sync_channel=True)
+    >>> recording = read_spikeglx(folder_path=r'path_to_folder_with_data')
     """
 
     NeoRawIOClass = "SpikeGLXRawIO"
 
     def __init__(
         self,
         folder_path,
-        load_sync_channel=False,
         stream_id=None,
         stream_name=None,
         all_annotations: bool = False,
         use_names_as_ids: bool = False,
     ):
 
-        neo_kwargs = self.map_to_neo_kwargs(folder_path, load_sync_channel=load_sync_channel)
+        neo_kwargs = self.map_to_neo_kwargs(folder_path)
         NeoBaseRecordingExtractor.__init__(
             self,
             stream_id=stream_id,
@@ -67,7 +61,7 @@ def __init__(
             **neo_kwargs,
         )
 
-        self._kwargs.update(dict(folder_path=str(Path(folder_path).absolute()), load_sync_channel=load_sync_channel))
+        self._kwargs.update(dict(folder_path=str(Path(folder_path).absolute())))
 
         stream_is_nidq = "nidq" in self.stream_id
         stream_is_one_box = "obx" in self.stream_id
@@ -83,7 +77,7 @@ def __init__(
 
         ap_meta_filename = meta_filename.replace(".lf", ".ap") if "lf" in self.stream_id else meta_filename
         ap_meta_file_exists = Path(ap_meta_filename).exists()
-        add_probe_properties = ap_meta_file_exists and not load_sync_channel
+        add_probe_properties = ap_meta_file_exists
 
         if add_probe_properties:
             probe = probeinterface.read_spikeglx(ap_meta_filename)
@@ -105,8 +99,8 @@ def __init__(
             warnings.warn(warning_message, UserWarning, stacklevel=2)
 
     @classmethod
-    def map_to_neo_kwargs(cls, folder_path, load_sync_channel=False):
-        neo_kwargs = {"dirname": str(folder_path), "load_sync_channel": load_sync_channel}
+    def map_to_neo_kwargs(cls, folder_path):
+        neo_kwargs = {"dirname": str(folder_path)}
         return neo_kwargs