Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit d176508

Browse filesBrowse files
authored
Merge pull request #14747 from anntzer/dpi_scale_trans
Deprecate AxisArtist.dpi_transform.
2 parents 4d927de + 4eb5dde commit d176508
Copy full SHA for d176508

File tree

Expand file treeCollapse file tree

2 files changed

+14
-16
lines changed
Filter options
Expand file treeCollapse file tree

2 files changed

+14
-16
lines changed

‎doc/api/api_changes_3.3/deprecations.rst

Copy file name to clipboardExpand all lines: doc/api/api_changes_3.3/deprecations.rst
+5Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -469,3 +469,8 @@ keyword-only, consistently with ``Colorbar``.
469469
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
470470
Passing ``None`` as either the ``radius`` or ``startangle`` of an `.Axes.pie`
471471
is deprecated; use the explicit defaults of 1 and 0, respectively, instead.
472+
473+
``AxisArtist.dpi_transform``
474+
~~~~~~~~~~~~~~~~~~~~~~~~~~~~
475+
... is deprecated. Scale ``Figure.dpi_scale_trans`` by 1/72 to achieve the
476+
same effect.

‎lib/mpl_toolkits/axisartist/axis_artist.py

Copy file name to clipboardExpand all lines: lib/mpl_toolkits/axisartist/axis_artist.py
+9-16Lines changed: 9 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -743,9 +743,10 @@ def __init__(self, axes,
743743

744744
if offset is None:
745745
offset = (0, 0)
746-
self.dpi_transform = Affine2D()
747-
self.offset_transform = ScaledTranslation(offset[0], offset[1],
748-
self.dpi_transform)
746+
self.offset_transform = ScaledTranslation(
747+
*offset,
748+
Affine2D().scale(1 / 72) # points to inches.
749+
+ self.axes.figure.dpi_scale_trans)
749750

750751
if axis_direction in ["left", "right"]:
751752
axis_name = "ytick"
@@ -772,6 +773,11 @@ def __init__(self, axes,
772773
self._axislabel_add_angle = 0.
773774
self.set_axis_direction(axis_direction)
774775

776+
@cbook.deprecated("3.3")
777+
@property
778+
def dpi_transform(self):
779+
return Affine2D().scale(1 / 72) + self.axes.figure.dpi_scale_trans
780+
775781
# axis direction
776782

777783
def set_axis_direction(self, axis_direction):
@@ -1181,12 +1187,7 @@ def set_label(self, s):
11811187
def get_tightbbox(self, renderer):
11821188
if not self.get_visible():
11831189
return
1184-
11851190
self._axis_artist_helper.update_lim(self.axes)
1186-
1187-
dpi_cor = renderer.points_to_pixels(1.)
1188-
self.dpi_transform.clear().scale(dpi_cor)
1189-
11901191
self._update_ticks(renderer)
11911192
self._update_label(renderer)
11921193
bb = [
@@ -1205,21 +1206,13 @@ def get_tightbbox(self, renderer):
12051206
@martist.allow_rasterization
12061207
def draw(self, renderer):
12071208
# docstring inherited
1208-
12091209
if not self.get_visible():
12101210
return
1211-
12121211
renderer.open_group(__name__, gid=self.get_gid())
1213-
12141212
self._axis_artist_helper.update_lim(self.axes)
1215-
1216-
dpi_cor = renderer.points_to_pixels(1.)
1217-
self.dpi_transform.clear().scale(dpi_cor)
1218-
12191213
self._draw_ticks(renderer)
12201214
self._draw_line(renderer)
12211215
self._draw_label(renderer)
1222-
12231216
renderer.close_group(__name__)
12241217

12251218
def toggle(self, all=None, ticks=None, ticklabels=None, label=None):

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.