actaully test latex compilation, fix a few bugs

Author: nschloe

test/helpers.py

@@ -30,7 +30,9 @@ def _unidiff_output(expected, actual):
     return "".join(diff)
 
 
-def assert_equality(plot, filename, **extra_get_tikz_code_args):
+def assert_equality(
+    plot, filename, assert_compilation=True, **extra_get_tikz_code_args
+):
     plot()
     code = tikzplotlib.get_tikz_code(
         include_disclaimer=False, float_format=".8g", **extra_get_tikz_code_args
@@ -42,11 +44,13 @@ def assert_equality(plot, filename, **extra_get_tikz_code_args):
         reference = f.read()
     assert reference == code, _unidiff_output(code, reference)
 
-    plot()
-    code = tikzplotlib.get_tikz_code(
-        include_disclaimer=False, standalone=True, **extra_get_tikz_code_args
-    )
-    assert _compile(code) is not None, code
+    if assert_compilation:
+        plot()
+        code = tikzplotlib.get_tikz_code(
+            include_disclaimer=False, standalone=True, **extra_get_tikz_code_args
+        )
+        plt.close()
+        assert _compile(code) is not None, code
 
 
 def _compile(code):

test/test_annotate_reference.tex

@@ -52,7 +52,7 @@
   text=black,
   rotate=0.0
 ]{arrowstyle};
-\draw (axis cs:0,1) ++(50px,-30px) node[
+\draw (axis cs:50,-30) node[
   scale=0.5,
   anchor=base west,
   text=black,

test/test_colorbars.py

@@ -75,8 +75,7 @@ def plot():
 
 
 def test():
-    assert_equality(plot, "test_colorbars_reference.tex")
-    return
+    assert_equality(plot, "test_colorbars_reference.tex", assert_compilation=False)
 
 
 if __name__ == "__main__":

test/test_legend_line_scatter_reference.tex

@@ -14,7 +14,7 @@
 ymin=-0.23928571, ymax=4.2392857,
 ytick style={color=black}
 ]
-\addplot [only marks, marker=*, draw=color0, fill=color0, colormap/viridis]
+\addplot [only marks, mark=*, draw=color0, fill=color0, colormap/viridis]
 table{%
 x                      y
 0 0

test/test_marker_reference.tex

@@ -15,7 +15,7 @@
 ymin=-1.1, ymax=1.1,
 ytick style={color=black}
 ]
-\addplot [only marks, marker=+, draw=black, fill=black, colormap/viridis]
+\addplot [only marks, mark=+, draw=black, fill=black, colormap/viridis]
 table{%
 x                      y
 0 0

test/test_scatter_reference.tex

@@ -16,7 +16,7 @@
 ymin=0.16291984, ymax=112.82609,
 ytick style={color=black}
 ]
-\addplot [only marks, marker=*, draw=color0, fill=color0, colormap/viridis]
+\addplot [only marks, mark=*, draw=color0, fill=color0, colormap/viridis]
 table{%
 x                      y
 0 10.447038

tikzplotlib/_path.py

@@ -176,7 +176,7 @@ def draw_pathcollection(data, obj):
         data, pgfplots_marker, marker_options = _mpl_marker2pgfp_marker(
             data, marker0, fc
         )
-        draw_options += [f"marker={pgfplots_marker}"] + marker_options
+        draw_options += [f"mark={pgfplots_marker}"] + marker_options
 
     # `only mark` plots don't need linewidth
     data, extra_draw_options = get_draw_options(data, obj, ec, fc, ls, None)

tikzplotlib/_save.py

@@ -218,20 +218,19 @@ def get_tikz_code(
         code += "\\end{tikzpicture}\n"
 
     if standalone:
-        # When using pdflatex, this is necessary:
-        # \\DeclareUnicodeCharacter{{2212}}{{−}}
-        code = """\\documentclass{{standalone}}
+        # When using pdflatex, \\DeclareUnicodeCharacter is necessary.
+        code = f"""\\documentclass{{standalone}}
 \\usepackage[utf8]{{inputenc}}
 \\usepackage{{pgfplots}}
+\\DeclareUnicodeCharacter{{2212}}{{−}}
 \\usepgfplotslibrary{{groupplots}}
 \\usepgfplotslibrary{{dateplot}}
+\\usetikzlibrary{{patterns}}
+\\usetikzlibrary{{shapes.arrows}}
 \\pgfplotsset{{compat=newest}}
 \\begin{{document}}
-{}
-\\end{{document}}\n""".format(
-            code
-        )
-
+{code}
+\\end{{document}}\n"""
     return code
 
 

tikzplotlib/_text.py

@@ -213,10 +213,10 @@ def _annotation(obj, data, content):
         x, y = ann_xytext
         unit = "pt"
         text_pos = f"{xy_pos} ++({x:{ff}}{unit},{y:{ff}}{unit})"
-    elif ann_textcoords == "offset pixels":
-        x, y = ann_xytext
-        unit = "px"
-        text_pos = f"{xy_pos} ++({x:{ff}}{unit},{y:{ff}}{unit})"
+    # elif ann_textcoords == "offset pixels":
+    #     x, y = ann_xytext
+    #     unit = "px"
+    #     text_pos = f"{xy_pos} ++({x:{ff}}{unit},{y:{ff}}{unit})"
     else:
         try:
             text_pos = _parse_annotation_coords(ff, ann_xycoords, ann_xytext)