FIX: completely remove QuadMesh cursor data

jklymak · jklymak · commit 38744e0e0304 · 2022-03-23T23:04:40.000+01:00
diff --git a/doc/api/next_api_changes/behavior/22254-DS.rst b/doc/api/next_api_changes/behavior/22254-DS.rst
diff --git a/doc/api/next_api_changes/behavior/22691-JMK.rst b/doc/api/next_api_changes/behavior/22691-JMK.rst
@@ -0,0 +1,4 @@
+QuadMesh cursor data removed
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+`.QuadMesh.get_cursor_data`, introduced in 3.5.1 is unreasonably slow for
+large meshes, so this method has been removed.
diff --git a/lib/matplotlib/collections.py b/lib/matplotlib/collections.py
@@ -1999,7 +1999,6 @@ def __init__(self, *args, **kwargs):
         self._shading = shading
         self._bbox = transforms.Bbox.unit()
         self._bbox.update_from_data_xy(self._coordinates.reshape(-1, 2))
-        self._show_cursor_data = False
         # super init delayed after own init because array kwarg requires
         # self._coordinates and self._shading
         super().__init__(**kwargs)
@@ -2196,27 +2195,3 @@ def draw(self, renderer):
         gc.restore()
         renderer.close_group(self.__class__.__name__)
         self.stale = False
-
-    def set_show_cursor_data(self, show_cursor_data):
-        """
-        Set whether cursor data should be shown.
-
-        Notes
-        -----
-        This is set to `False` by default for new quad meshes. Showing cursor
-        data can have significant performance impacts for large meshes.
-        """
-        self._show_cursor_data = show_cursor_data
-
-    def get_show_cursor_data(self):
-        return self._show_cursor_data
-
-    def get_cursor_data(self, event):
-        if not self._show_cursor_data:
-            return
-        contained, info = self.contains(event)
-        if len(info["ind"]) == 1:
-            ind, = info["ind"]
-            return self.get_array()[ind]
-        else:
-            return None
diff --git a/lib/matplotlib/tests/test_collections.py b/lib/matplotlib/tests/test_collections.py
@@ -7,7 +7,6 @@
 
 import matplotlib as mpl
 import matplotlib.pyplot as plt
-from matplotlib.backend_bases import MouseEvent
 import matplotlib.collections as mcollections
 import matplotlib.colors as mcolors
 import matplotlib.transforms as mtransforms
@@ -1039,26 +1038,6 @@ def test_array_wrong_dimensions():
     pc.update_scalarmappable()
 
 
-def test_quadmesh_cursor_data():
-    fig, ax = plt.subplots()
-    *_, qm = ax.hist2d(
-        np.arange(11)**2, 100 + np.arange(11)**2)  # width-10 bins
-
-    x, y = ax.transData.transform([1, 101])
-    event = MouseEvent('motion_notify_event', fig.canvas, x, y)
-
-    assert qm.get_show_cursor_data() is False
-    assert qm.get_cursor_data(event) is None
-
-    qm.set_show_cursor_data(True)
-    assert qm.get_cursor_data(event) == 4  # (0**2, 1**2, 2**2, 3**2)
-
-    # Outside the quadmesh bounds
-    x, y = ax.transData.transform([-1, 101])
-    event = MouseEvent('motion_notify_event', fig.canvas, x, y)
-    assert qm.get_cursor_data(event) is None
-
-
 def test_get_segments():
     segments = np.tile(np.linspace(0, 1, 256), (2, 1)).T
     lc = LineCollection([segments])
