Unify pickradius arguments and checking

Author: oscargus

lib/matplotlib/axis.py

@@ -640,8 +640,7 @@ def __str__(self):
         return "{}({},{})".format(
             type(self).__name__, *self.axes.transAxes.transform((0, 0)))
 
-    @_api.make_keyword_only("3.6", name="pickradius")
-    def __init__(self, axes, pickradius=15):
+    def __init__(self, axes, *, pickradius=15):
         """
         Parameters
         ----------

lib/matplotlib/collections.py

@@ -433,7 +433,6 @@ def draw(self, renderer):
         renderer.close_group(self.__class__.__name__)
         self.stale = False
 
-    @_api.rename_parameter("3.6", "pr", "pickradius")
     def set_pickradius(self, pickradius):
         """
         Set the pick radius used for containment tests.
@@ -443,6 +442,9 @@ def set_pickradius(self, pickradius):
         pickradius : float
             Pick radius, in points.
         """
+        if not isinstance(pickradius, Number):
+            raise ValueError(
+                f"pickradius must be a number, not {pickradius!r}")
         self._pickradius = pickradius
 
     def get_pickradius(self):

lib/matplotlib/lines.py

@@ -404,7 +404,7 @@ def __init__(self, xdata, ydata,
         # update kwargs before updating data to give the caller a
         # chance to init axes (and hence unit support)
         self._internal_update(kwargs)
-        self._pickradius = pickradius
+        self.pickradius = pickradius
         self.ind_offset = 0
         if (isinstance(self._picker, Number) and
                 not isinstance(self._picker, bool)):
@@ -502,7 +502,6 @@ def get_pickradius(self):
         """
         return self._pickradius
 
-    @_api.rename_parameter("3.6", "d", "pickradius")
     def set_pickradius(self, pickradius):
         """
         Set the pick radius used for containment tests.