Merge pull request #430 from BiAPoL/call-replot-by-signal

Fix plot clearing

Author: jo-mueller

src/napari_clusters_plotter/_new_plotter_widget.py

@@ -60,7 +60,7 @@ def __init__(self, napari_viewer):
                 self.overlay_colormap_plot
             ),
         }
-        self._replot()
+        self.plot_needs_update.emit()
 
     def _napari_to_mpl_cmap(self, colormap_name):
         return LinearSegmentedColormap.from_list(
@@ -352,7 +352,7 @@ def _on_plot_type_changed(self):
             self.plotting_widget.active_artist.overlay_colormap = (
                 cat10_mod_cmap
             )
-        self._replot()
+        self.plot_needs_update.emit()
 
     def _on_overlay_colormap_changed(self):
         colormap_name = self.overlay_colormap_plot
@@ -363,21 +363,21 @@ def _on_overlay_colormap_changed(self):
         self.colormap_reference[(False, "SCATTER")] = self._napari_to_mpl_cmap(
             colormap_name
         )
-        self._replot()
+        self.plot_needs_update.emit()
 
     def _on_histogram_colormap_changed(self):
-        self._replot()
+        self.plot_needs_update.emit()
 
     def _checkbox_status_changed(self):
-        self._replot()
+        self.plot_needs_update.emit()
 
     def _on_bin_auto_toggled(self, state: bool):
         """
         Called when the automatic bin checkbox is toggled.
         Enables or disables the bin number box accordingly.
         """
         self.control_widget.n_bins_box.setEnabled(not state)
-        self._replot()
+        self.plot_needs_update.emit()
 
     # Connecting the widgets to actual object variables:
     # using getters and setters for flexibility
@@ -437,7 +437,7 @@ def x_axis(self):
     @x_axis.setter
     def x_axis(self, column: str):
         self.control_widget.x_axis_box.setCurrentText(column)
-        self._replot()
+        self.plot_needs_update.emit()
 
     @property
     def y_axis(self):
@@ -446,7 +446,7 @@ def y_axis(self):
     @y_axis.setter
     def y_axis(self, column: str):
         self.control_widget.y_axis_box.setCurrentText(column)
-        self._replot()
+        self.plot_needs_update.emit()
 
     @property
     def hue_axis(self):
@@ -744,3 +744,4 @@ def _reset(self):
         # )
         self._update_layer_colors(use_color_indices=False)
         self.control_widget.hue_box.setCurrentText("MANUAL_CLUSTER_ID")
+        self.plot_needs_update.emit()