Fix double refresh based on time info

Author: alejoe91

spikeinterface_gui/basescatterview.py

@@ -32,6 +32,7 @@ def __init__(self, spike_data, y_label, controller=None, parent=None, backend="q
         self._lasso_vertices = {segment_index: None for segment_index in range(controller.num_segments)}
         # this is used in panel
         self._current_selected = 0
+        self._block_auto_refresh_and_notify = False
 
         ViewBase.__init__(self, controller=controller, parent=parent,  backend=backend)
 
@@ -164,11 +165,14 @@ def on_unit_visibility_changed(self):
                 self._current_selected = self.controller.get_indices_spike_selected().size
         self.refresh()
 
-    def on_time_info_updated(self):
-        return self.refresh()
+    def _qt_on_time_info_updated(self):
+        if self.combo_seg.currentIndex() != self.controller.get_time()[1]:
+            self._block_auto_refresh_and_notify = True
+            self.refresh()
+            self._block_auto_refresh_and_notify = False
 
     def on_use_times_updated(self):
-        return self.refresh()
+        self.refresh()
 
     ## QT zone ##
     def _qt_make_layout(self):
@@ -242,8 +246,9 @@ def _qt_on_spike_selection_changed(self):
     def _qt_change_segment(self):
         segment_index = self.combo_seg.currentIndex()
         self.controller.set_time(segment_index=segment_index)
-        self.refresh()
-        self.notify_time_info_updated()
+        if not self._block_auto_refresh_and_notify:
+            self.refresh()
+            self.notify_time_info_updated()
 
     def _qt_refresh(self):
         from .myqt import QT

spikeinterface_gui/spikerateview.py

@@ -11,12 +11,16 @@ class SpikeRateView(ViewBase):
 
     def __init__(self, controller=None, parent=None, backend="qt"):
         ViewBase.__init__(self, controller=controller, parent=parent,  backend=backend)
+        self._block_auto_refresh_and_notify = False
 
     def _on_settings_changed(self):
         self.refresh()
 
-    def on_time_info_updated(self):
-        self.refresh()
+    def _qt_on_time_info_updated(self):
+        if self.combo_seg.currentIndex() != self.controller.get_time()[1]:
+            self._block_auto_refresh_and_notify = True
+            self.refresh()
+            self._block_auto_refresh_and_notify = False
 
     def on_use_times_updated(self):
         self.refresh()
@@ -46,8 +50,9 @@ def _qt_make_layout(self):
     def _qt_change_segment(self):
         segment_index = self.combo_seg.currentIndex()
         self.controller.set_time(segment_index=segment_index)
-        self.refresh()
-        self.notify_time_info_updated()
+        if not self._block_auto_refresh_and_notify:
+            self.refresh()
+            self.notify_time_info_updated()
 
     def _qt_refresh(self):
         import pyqtgraph as pg