Inkscape shell mode.

anntzer · anntzer · commit 2a25286e8d09 · 2017-03-11T23:43:48.000-08:00
diff --git a/lib/matplotlib/testing/compare.py b/lib/matplotlib/testing/compare.py
@@ -7,9 +7,14 @@
 
 import six
 
+import atexit
+import functools
 import hashlib
+import itertools
 import os
+import shlex
 import shutil
+import sys
 
 import numpy as np
 
@@ -128,6 +133,67 @@ def convert(old, new):
     return convert
 
 
+class _SVGConverter(object):
+    def __init__(self):
+        self._proc = None
+        # We cannot rely on the GC to trigger `__del__` at exit because
+        # other modules (e.g. `subprocess`) may already have their globals
+        # set to `None`, which make `proc.communicate` or `proc.terminate`
+        # fail.  By relying on `atexit` we ensure the destructor runs before
+        # `None`-setting occurs.
+        atexit.register(self.__del__)
+
+    def _read_to_terminator(self):
+        stream = iter(functools.partial(self._proc.stdout.read, 1), "")
+        terminator = tuple("\n>")
+        n = len(terminator)
+        its = itertools.tee(stream, n)
+        for i, it in enumerate(its):
+            next(itertools.islice(it, i, i), None)  # Advance `it` by `i`.
+        while True:
+            window = tuple(map(next, its))
+            if (len(window) != n or window == terminator):
+                # First case is to running out of data -- one of the `next(it)`
+                # raises StopIteration, so the tuple is shorter.
+                # Second case is to successful reading until terminator.
+                break
+
+    def __call__(self, orig, dest):
+        if (not self._proc  # First run.
+                or self._proc.poll()):  # Inkscape terminated, e.g. crashed.
+            env = os.environ.copy()
+            # If one passes e.g. a png file to Inkscape, it will try to
+            # query the user for conversion options via a GUI (even with
+            # `--without-gui`).  Unsetting `DISPLAY` prevents this (and causes
+            # GTK to crash and Inkscape to terminate, but that'll just be
+            # reported as a regular exception below).
+            env["DISPLAY"] = ""
+            # Do not load any user options.
+            env["INKSCAPE_PROFILE_DIR"] = os.devnull
+            self._proc = subprocess.Popen(
+                [str("inkscape"), "--without-gui", "--shell"],
+                env=env, stdin=subprocess.PIPE,
+                stdout=subprocess.PIPE, stderr=subprocess.PIPE,
+                universal_newlines=True)
+            self._read_to_terminator()
+        # Inkscape uses glib's `g_shell_parse_argv`, which has a consistent
+        # behavior across platforms.
+        self._proc.stdin.write(
+            "{} --export-png={}\n".format(*map(shlex.quote, [orig, dest])))
+        self._proc.stdin.flush()
+        self._read_to_terminator()  # Ensure that the conversion is done.
+        if self._proc.poll():  # Inkscape exits on error.
+            raise ImageComparisonFailure(self._proc.stderr.read())
+
+    def __del__(self):
+        if self._proc:
+            if self._proc.poll() is None:  # Not exited yet.
+                self._proc.communicate("quit\n")
+                self._proc.wait()
+            self._proc.stdin.close()
+            self._proc.stdout.close()
+
+
 def _update_converter():
     gs, gs_v = matplotlib.checkdep_ghostscript()
     if gs_v is not None:
@@ -138,9 +204,7 @@ def cmd(old, new):
         converter['eps'] = make_external_conversion_command(cmd)
 
     if matplotlib.checkdep_inkscape() is not None:
-        def cmd(old, new):
-            return [str('inkscape'), '-z', old, '--export-png', new]
-        converter['svg'] = make_external_conversion_command(cmd)
+        converter['svg'] = _SVGConverter()
 
 
 #: A dictionary that maps filename extensions to functions which
@@ -363,9 +427,8 @@ def save_diff_image(expected, actual, output):
         actual, actualImage, expected, expectedImage)
     expectedImage = np.array(expectedImage).astype(float)
     actualImage = np.array(actualImage).astype(float)
-    assert expectedImage.ndim == actualImage.ndim
     assert expectedImage.shape == actualImage.shape
-    absDiffImage = abs(expectedImage - actualImage)
+    absDiffImage = np.abs(expectedImage - actualImage)
 
     # expand differences in luminance domain
     absDiffImage *= 255 * 10
