Use _repr_png_ with other suggested PR changes.

bdice · bdice · commit 46e810e89c6d · 2020-07-11T16:25:38.000-05:00
diff --git a/lib/matplotlib/backend_bases.py b/lib/matplotlib/backend_bases.py
@@ -1595,44 +1595,24 @@ def __init__(self, figure):
         self.toolbar = None  # NavigationToolbar2 will set me
         self._is_idle_drawing = False
 
-    def _repr_html_(self):
-        # Defer to IPython to handle html output if possible
+    def _repr_png_(self):
+        # Defer to IPython to handle output if possible.
         if 'IPython' in sys.modules:
-            import IPython
-            ip = IPython.get_ipython()
-            # Check whether %matplotlib was run. Is there a better way?
-            ib_list = [c for c in ip.configurables 
-                       if 'InlineBackend' in type(c).__name__]
-            if ib_list:
+            from IPython.core.pylabtools import configure_inline_support
+            # Check whether %matplotlib was run.
+            if hasattr(configure_inline_support, 'current_backend'):
                 return
-            
-        fmt = self.get_default_filetype()
-
-        kw = {
-            "format":fmt,
-            "facecolor":self.figure.get_facecolor(),
-            "edgecolor":self.figure.get_edgecolor(),
-            "dpi":self.figure.dpi,
-            "bbox_inches":self.figure.bbox_inches
-        }
-
-        bytes_io = io.BytesIO()
-        self.print_figure(bytes_io, **kw)
-        raw_bytes = bytes_io.getvalue()
-
-        from base64 import b64encode
-        data = b64encode(raw_bytes).decode()
-
-        if fmt == 'svg':
-            return raw_bytes.decode()
-        elif fmt == 'png':
-            return f'<img src="data:image/png;base64, {data}" />'
-        elif fmt == 'pdf':
-            w, h = self.figure.get_size_inches()
-            w, h = w * self.figure.dpi, h * self.figure.dpi
-            return f'<embed width="{w}" height="{h}" src="data:application/pdf;base64, {data}">'
-        elif fmt == 'jpg':
-            return f'<img src="data:image/jpeg;base64, {data}" />'
+
+        png_bytes = io.BytesIO()
+        self.print_figure(
+            png_bytes,
+            format='png',
+            facecolor=self.figure.get_facecolor(),
+            edgecolor=self.figure.get_edgecolor(),
+            dpi=self.figure.dpi,
+            bbox_inches=self.figure.bbox_inches
+        )
+        return png_bytes.getvalue()
 
     @classmethod
     @functools.lru_cache()
diff --git a/lib/matplotlib/figure.py b/lib/matplotlib/figure.py
@@ -364,12 +364,8 @@ def __init__(self,
         # list of child gridspecs for this figure
         self._gridspecs = []
 
-    # TODO: I'd like to dynamically add the _repr_html_ method
-    # to the figure in the right context, but then IPython doesn't
-    # use it, for some reason.
-
-    def _repr_html_(self):
-        return self.canvas._repr_html_()
+    def _repr_png_(self):
+        return self.canvas._repr_png_()
 
     def show(self, warn=True):
         """
