Only do pchanged and set stale when value changes

oscargus · oscargus · commit 59dbd1e5fada · 2023-07-19T02:51:58.000+02:00
diff --git a/lib/matplotlib/artist.py b/lib/matplotlib/artist.py
@@ -353,7 +353,7 @@ def get_tightbbox(self, renderer=None):
 
         Parameters
         ----------
-        renderer : `.RendererBase` subclass
+        renderer : `~matplotlib.backend_bases.RendererBase` subclass, optional
             renderer that will be used to draw the figures (i.e.
             ``fig.canvas.get_renderer()``)
 
@@ -440,7 +440,7 @@ def set_transform(self, t):
 
         Parameters
         ----------
-        t : `.Transform`
+        t : `~matplotlib.transforms.Transform`
         """
         self._transform = t
         self._transformSet = True
@@ -733,7 +733,7 @@ def set_figure(self, fig):
 
         Parameters
         ----------
-        fig : `.Figure`
+        fig : `~matplotlib.figure.Figure`
         """
         # if this is a no-op just return
         if self.figure is fig:
@@ -757,16 +757,17 @@ def set_clip_box(self, clipbox):
 
         Parameters
         ----------
-        clipbox : `.Bbox`
+        clipbox : `~matplotlib.transforms.Bbox`
 
-            Typically would be created from a `.TransformedBbox`. For
+            Will typically be created from a `.TransformedBbox`. For
             instance ``TransformedBbox(Bbox([[0, 0], [1, 1]]), ax.transAxes)``
             is the default clipping for an artist added to an Axes.
 
         """
-        self.clipbox = clipbox
-        self.pchanged()
-        self.stale = True
+        if clipbox != self.clipbox:
+            self.clipbox = clipbox
+            self.pchanged()
+            self.stale = True
 
     def set_clip_path(self, path, transform=None):
         """
@@ -987,7 +988,7 @@ def draw(self, renderer):
 
         Parameters
         ----------
-        renderer : `.RendererBase` subclass.
+        renderer : `~matplotlib.backend_bases.RendererBase` subclass.
 
         Notes
         -----
@@ -1011,9 +1012,10 @@ def set_alpha(self, alpha):
                 f'alpha must be numeric or None, not {type(alpha)}')
         if alpha is not None and not (0 <= alpha <= 1):
             raise ValueError(f'alpha ({alpha}) is outside 0-1 range')
-        self._alpha = alpha
-        self.pchanged()
-        self.stale = True
+        if alpha != self._alpha:
+            self._alpha = alpha
+            self.pchanged()
+            self.stale = True
 
     def _set_alpha_for_array(self, alpha):
         """
@@ -1046,9 +1048,10 @@ def set_visible(self, b):
         ----------
         b : bool
         """
-        self._visible = b
-        self.pchanged()
-        self.stale = True
+        if b != self._visible:
+            self._visible = b
+            self.pchanged()
+            self.stale = True
 
     def set_animated(self, b):
         """
@@ -1096,9 +1099,11 @@ def set_label(self, s):
         s : object
             *s* will be converted to a string by calling `str`.
         """
-        self._label = str(s) if s is not None else None
-        self.pchanged()
-        self.stale = True
+        label = str(s) if s is not None else None
+        if label != self._label:
+            self._label = label
+            self.pchanged()
+            self.stale = True
 
     def get_zorder(self):
         """Return the artist's zorder."""
@@ -1115,9 +1120,10 @@ def set_zorder(self, level):
         """
         if level is None:
             level = self.__class__.zorder
-        self.zorder = level
-        self.pchanged()
-        self.stale = True
+        if level != self.zorder:
+            self.zorder = level
+            self.pchanged()
+            self.stale = True
 
     @property
     def sticky_edges(self):
