Add checks for start/end_frames

src/spikeinterface/core/baserecording.py

@@ -331,8 +331,12 @@ def get_traces(
         segment_index = self._check_segment_index(segment_index)
         channel_indices = self.ids_to_indices(channel_ids, prefer_slice=True)
         rs = self._recording_segments[segment_index]
-        start_frame = int(max(0, start_frame)) if start_frame is not None else 0
+        start_frame = int(start_frame) if start_frame is not None else 0
         end_frame = int(min(end_frame, rs.get_num_samples())) if end_frame is not None else rs.get_num_samples()
+        if start_frame < 0:
+            raise ValueError("start_frame cannot be negative")
+        if start_frame > end_frame:
+            raise ValueError("start_frame cannot be greater than end_frame")
         traces = rs.get_traces(start_frame=start_frame, end_frame=end_frame, channel_indices=channel_indices)
         if order is not None:
             assert order in ["C", "F"]