Fix tests and array_annotations for SYNC channel

Author: alejoe91

neo/rawio/openephysbinaryrawio.py

@@ -513,12 +513,17 @@ def _parse_header(self):
                             if has_sync_trace:
                                 values = values[:-1]
 
-                            neural_channels = [ch for ch in info["channels"] if "ADC" not in ch["channel_name"]]
-                            num_neural_channels = len(neural_channels)
-                            if is_neural_stream:
-                                values = values[:num_neural_channels]
-                            else:
-                                values = values[num_neural_channels:]
+                            if "SYNC" in stream_name and not self.load_sync_channel:
+                                # This is the sync stream, we only keep the last value
+                                values = values[-1:]
+
+                            if "OneBox" not in info["stream_name"]:
+                                neural_channels = [ch for ch in info["channels"] if "ADC" not in ch["channel_name"]]
+                                num_neural_channels = len(neural_channels)
+                                if is_neural_stream:
+                                    values = values[:num_neural_channels]
+                                else:
+                                    values = values[num_neural_channels:]
 
                             sig_ann["__array_annotations__"][key] = values
 

neo/test/rawiotest/test_openephysbinaryrawio.py

@@ -21,10 +21,11 @@ class TestOpenEphysBinaryRawIO(BaseTestRawIO, unittest.TestCase):
     ]
 
     def test_sync(self):
-        rawio_with_sync = OpenEphysBinaryRawIO(
-            self.get_local_path("openephysbinary/v0.6.x_neuropixels_with_sync"), load_sync_channel=True
-        )
-        rawio_with_sync.parse_header()
+        with self.assertWarns(DeprecationWarning):
+            rawio_with_sync = OpenEphysBinaryRawIO(
+                self.get_local_path("openephysbinary/v0.6.x_neuropixels_with_sync"), load_sync_channel=True
+            )
+            rawio_with_sync.parse_header()
         stream_name = [s_name for s_name in rawio_with_sync.header["signal_streams"]["name"] if "AP" in s_name][0]
         stream_index = list(rawio_with_sync.header["signal_streams"]["name"]).index(stream_name)
 
@@ -70,10 +71,11 @@ def test_sync_channel_access(self):
     def test_no_sync(self):
         # requesting sync channel when there is none raises an error
         with self.assertRaises(ValueError):
-            rawio_no_sync = OpenEphysBinaryRawIO(
-                self.get_local_path("openephysbinary/v0.6.x_neuropixels_multiexp_multistream"), load_sync_channel=True
-            )
-            rawio_no_sync.parse_header()
+            with self.assertWarns(DeprecationWarning):
+                rawio_no_sync = OpenEphysBinaryRawIO(
+                    self.get_local_path("openephysbinary/v0.6.x_neuropixels_multiexp_multistream"), load_sync_channel=True
+                )
+                rawio_no_sync.parse_header()
 
     def test_missing_folders(self):
         # missing folders should raise an error