FIX: fix figure.set_dpi when pixel ratio not 1

jklymak · jklymak · commit 211f84f4f307 · 2018-05-11T10:17:39.000-07:00
diff --git a/lib/matplotlib/backends/backend_qt5.py b/lib/matplotlib/backends/backend_qt5.py
@@ -229,7 +229,6 @@ def __init__(self, figure):
         self.figure = figure
         # We don't want to scale up the figure DPI more than once.
         # Note, we don't handle a signal for changing DPI yet.
-        figure._original_dpi = figure.dpi
         self._update_figure_dpi()
         # In cases with mixed resolution displays, we need to be careful if the
         # dpi_ratio changes - in this case we need to resize the canvas
diff --git a/lib/matplotlib/figure.py b/lib/matplotlib/figure.py
@@ -357,6 +357,7 @@ def __init__(self,
         self.dpi_scale_trans = Affine2D().scale(dpi, dpi)
         # do not use property as it will trigger
         self._dpi = dpi
+        self._original_dpi = dpi
         self.bbox = TransformedBbox(self.bbox_inches, self.dpi_scale_trans)
 
         self.frameon = frameon
@@ -964,13 +965,37 @@ def set_facecolor(self, color):
         """
         self.patch.set_facecolor(color)
 
-    def set_dpi(self, val):
+    def set_dpi(self, dpi):
         """
-        Set the resolution of the figure in dots-per-inch.
+        Set the nominal resolution of the figure in dots-per-inch.
 
-        .. ACCEPTS: float
-        """
-        self.dpi = val
+        Parameters
+        ----------
+        dpi : float
+
+        Notes
+        -----
+        Some backends have the concept of "hi-dpi" displays (or "Retina")
+        where the resolution is doubled, but dimensions are traditionally
+        specified as if the resolution were not doubled.  Some Matplotlib
+        GUI backends respect this doubling (i.e. Qt5Agg), but Matplotlib still
+        specifies dimensions and dpi to matplotlib as if the resolution
+        were not doubled. The default figure
+        value is nominally 100 dpi, but displays that are deignated "hi-dpi"
+        will internally double this to 200 dpi.  We keep the "nominal" dpi
+        because some computer set ups have one display at normal dpi, and a
+        second at hi-dpi, so a nominal resolution must be stored to stop
+        figures from doubling or halving is size when moved between displays.
+        """
+
+        # some backends set self._dpi to be a ratio times
+        # self._original_dpi: 
+        ratio = self._dpi / self._original_dpi
+
+        # _original_dpi is the nominal dpi before any hi-dpi changes.
+        self._original_dpi = dpi
+        # calls _set_dpi with the actual display dpi...
+        self.dpi = dpi * ratio
         self.stale = True
 
     def set_figwidth(self, val, forward=True):
