Skip to content

Commit c1731b3

Browse files
committed
Remove Axes internals deprecated in 3.3.
1 parent fc92d96 commit c1731b3

File tree

5 files changed

+24
-64
lines changed

5 files changed

+24
-64
lines changed

doc/api/axes_api.rst

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,6 @@ Axis limits and direction
295295
Axes.get_ylim
296296

297297
Axes.update_datalim
298-
Axes.update_datalim_bounds
299298

300299
Axes.set_xbound
301300
Axes.get_xbound

doc/api/next_api_changes/removals/20331-ES.rst

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,20 @@
1+
Signatures of `.Artist.draw` and `.Axes.draw`
2+
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
3+
The *inframe* parameter to `.Axes.draw` has been removed. Use
4+
`.Axes.redraw_in_frame` instead.
5+
6+
Omitting the *renderer* parameter to `.Axes.draw` is no longer supported.
7+
Use ``axes.draw_artist(axes)`` instead.
8+
9+
These changes make the signature of the ``draw`` (``artist.draw(renderer)``)
10+
method consistent across all artists; thus, additional parameters to
11+
`.Artist.draw` have also been removed.
12+
13+
``Axes.update_datalim_bounds``
14+
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
15+
This method has been removed. Use
16+
``ax.dataLim.set(Bbox.union([ax.dataLim, bounds]))`` instead.
17+
118
``{,Symmetrical}LogScale.{,Inverted}LogTransform``
219
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
320
``LogScale.LogTransform``, ``LogScale.InvertedLogTransform``,

lib/matplotlib/artist.py

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,8 @@ def allow_rasterization(draw):
2727
renderer.
2828
"""
2929

30-
# Axes has a second (deprecated) argument inframe for its draw method.
31-
# args and kwargs are deprecated, but we don't wrap this in
32-
# _api.delete_parameter for performance; the relevant deprecation
33-
# warning will be emitted by the inner draw() call.
3430
@wraps(draw)
35-
def draw_wrapper(artist, renderer, *args, **kwargs):
31+
def draw_wrapper(artist, renderer):
3632
try:
3733
if artist.get_rasterized():
3834
if renderer._raster_depth == 0 and not renderer._rasterizing:
@@ -49,7 +45,7 @@ def draw_wrapper(artist, renderer, *args, **kwargs):
4945
if artist.get_agg_filter() is not None:
5046
renderer.start_filter()
5147

52-
return draw(artist, renderer, *args, **kwargs)
48+
return draw(artist, renderer)
5349
finally:
5450
if artist.get_agg_filter() is not None:
5551
renderer.stop_filter(artist.get_agg_filter())
@@ -929,9 +925,7 @@ def set_agg_filter(self, filter_func):
929925
self._agg_filter = filter_func
930926
self.stale = True
931927

932-
@_api.delete_parameter("3.3", "args")
933-
@_api.delete_parameter("3.3", "kwargs")
934-
def draw(self, renderer, *args, **kwargs):
928+
def draw(self, renderer):
935929
"""
936930
Draw the Artist (and its children) using the given renderer.
937931

lib/matplotlib/axes/_base.py

Lines changed: 2 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -2459,19 +2459,6 @@ def update_datalim(self, xys, updatex=True, updatey=True):
24592459
updatex=updatex, updatey=updatey)
24602460
self.ignore_existing_data_limits = False
24612461

2462-
@_api.deprecated(
2463-
"3.3", alternative="ax.dataLim.set(Bbox.union([ax.dataLim, bounds]))")
2464-
def update_datalim_bounds(self, bounds):
2465-
"""
2466-
Extend the `~.Axes.datalim` Bbox to include the given
2467-
`~matplotlib.transforms.Bbox`.
2468-
2469-
Parameters
2470-
----------
2471-
bounds : `~matplotlib.transforms.Bbox`
2472-
"""
2473-
self.dataLim.set(mtransforms.Bbox.union([self.dataLim, bounds]))
2474-
24752462
def _process_unit_info(self, datasets=None, kwargs=None, *, convert=True):
24762463
"""
24772464
Set axis units based on *datasets* and *kwargs*, and optionally apply
@@ -3011,17 +2998,8 @@ def _update_title_position(self, renderer):
30112998

30122999
# Drawing
30133000
@martist.allow_rasterization
3014-
@_api.delete_parameter(
3015-
"3.3", "inframe", alternative="Axes.redraw_in_frame()")
3016-
def draw(self, renderer=None, inframe=False):
3001+
def draw(self, renderer):
30173002
# docstring inherited
3018-
if renderer is None:
3019-
_api.warn_deprecated(
3020-
"3.3", message="Support for not passing the 'renderer' "
3021-
"parameter to Axes.draw() is deprecated since %(since)s and "
3022-
"will be removed %(removal)s. Use axes.draw_artist(axes) "
3023-
"instead.")
3024-
renderer = self.figure._cachedRenderer
30253003
if renderer is None:
30263004
raise RuntimeError('No renderer defined')
30273005
if not self.get_visible():
@@ -3054,15 +3032,10 @@ def draw(self, renderer=None, inframe=False):
30543032

30553033
self._update_title_position(renderer)
30563034

3057-
if not self.axison or inframe:
3035+
if not self.axison:
30583036
for _axis in self._get_axis_list():
30593037
artists.remove(_axis)
30603038

3061-
if inframe:
3062-
artists.remove(self.title)
3063-
artists.remove(self._left_title)
3064-
artists.remove(self._right_title)
3065-
30663039
if not self.figure.canvas.is_saving():
30673040
artists = [
30683041
a for a in artists

lib/matplotlib/projections/polar.py

Lines changed: 2 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -246,10 +246,6 @@ def __call__(self):
246246
else:
247247
return np.deg2rad(self.base())
248248

249-
@_api.deprecated("3.3")
250-
def pan(self, numsteps):
251-
return self.base.pan(numsteps)
252-
253249
def refresh(self):
254250
# docstring inherited
255251
return self.base.refresh()
@@ -258,10 +254,6 @@ def view_limits(self, vmin, vmax):
258254
vmin, vmax = np.rad2deg((vmin, vmax))
259255
return np.deg2rad(self.base.view_limits(vmin, vmax))
260256

261-
@_api.deprecated("3.3")
262-
def zoom(self, direction):
263-
return self.base.zoom(direction)
264-
265257

266258
class ThetaTick(maxis.XTick):
267259
"""
@@ -437,19 +429,6 @@ def __call__(self):
437429
return [tick for tick in self.base() if tick > rorigin]
438430
return self.base()
439431

440-
@_api.deprecated("3.3")
441-
def pan(self, numsteps):
442-
return self.base.pan(numsteps)
443-
444-
@_api.deprecated("3.3")
445-
def zoom(self, direction):
446-
return self.base.zoom(direction)
447-
448-
@_api.deprecated("3.3")
449-
def refresh(self):
450-
# docstring inherited
451-
return self.base.refresh()
452-
453432
def nonsingular(self, vmin, vmax):
454433
# docstring inherited
455434
return ((0, 1) if (vmin, vmax) == (-np.inf, np.inf) # Init. limits.
@@ -951,9 +930,7 @@ def get_yaxis_text2_transform(self, pad):
951930
pad_shift = _ThetaShift(self, pad, 'min')
952931
return self._yaxis_text_transform + pad_shift, 'center', halign
953932

954-
@_api.delete_parameter("3.3", "args")
955-
@_api.delete_parameter("3.3", "kwargs")
956-
def draw(self, renderer, *args, **kwargs):
933+
def draw(self, renderer):
957934
self._unstale_viewLim()
958935
thetamin, thetamax = np.rad2deg(self._realViewLim.intervalx)
959936
if thetamin > thetamax:
@@ -997,7 +974,7 @@ def draw(self, renderer, *args, **kwargs):
997974
self.yaxis.reset_ticks()
998975
self.yaxis.set_clip_path(self.patch)
999976

1000-
super().draw(renderer, *args, **kwargs)
977+
super().draw(renderer)
1001978

1002979
def _gen_axes_patch(self):
1003980
return mpatches.Wedge((0.5, 0.5), 0.5, 0.0, 360.0)

0 commit comments

Comments
 (0)