Issue #298; "table_row_sep" option was not working properly; the separator was used generating the table,

but the table setting itself was not adjusted to include the updated setting because the code to do so was
 nested in an (unrelated) conditional.  Further, the code in that unrelated conditional used the wrong key
value anyway.  Resolved by moving check outside unrelated conditional, fixed broken key, strip trailing
newline in table declaration.  Added test cases to check both with and without the separator (note there
are test failures, but they were failing before I made any changes; failures remain the same after my change).

Author: fsgeek

test/test_table_row_sep.py

@@ -0,0 +1,20 @@
+import matplotlib.pyplot as plt
+
+from helpers import assert_equality
+
+
+def plot():
+    a = [pow(10, i) for i in range(10)]
+    fig = plt.figure()
+    ax = fig.add_subplot(1, 1, 1)
+    ax.semilogy(a, color="blue", lw=0.25)
+
+    plt.grid(b=True, which="major", color="g", linestyle="-", linewidth=0.25)
+    plt.grid(b=True, which="minor", color="r", linestyle="--", linewidth=0.5)
+    return fig
+
+
+def test():
+    assert_equality(plot, __file__[:-3] + "_reference.tex", table_row_sep='\\\\\n')
+    assert_equality(plot, __file__[:-3] + "_reference2.tex")
+    return

test/test_table_row_sep_reference.tex

@@ -0,0 +1,34 @@
+\begin{tikzpicture}
+
+\begin{axis}[
+log basis y={10},
+tick align=outside,
+tick pos=left,
+x grid style={green!50.0!black},
+xmajorgrids,
+xmin=-0.45, xmax=9.45,
+xminorgrids,
+xtick style={color=black},
+y grid style={green!50.0!black},
+ymajorgrids,
+ymin=0.354813389233575, ymax=2818382931.26445,
+yminorgrids,
+ymode=log,
+ytick style={color=black}
+]
+\addplot [ultra thin, blue]
+table [row sep=\\] {%
+0 1\\
+1 10\\
+2 100\\
+3 1000\\
+4 10000\\
+5 100000\\
+6 1000000\\
+7 10000000\\
+8 100000000\\
+9 1000000000\\
+};
+\end{axis}
+
+\end{tikzpicture}

test/test_table_row_sep_reference2.tex

@@ -0,0 +1,34 @@
+\begin{tikzpicture}
+
+\begin{axis}[
+log basis y={10},
+tick align=outside,
+tick pos=left,
+x grid style={green!50.0!black},
+xmajorgrids,
+xmin=-0.45, xmax=9.45,
+xminorgrids,
+xtick style={color=black},
+y grid style={green!50.0!black},
+ymajorgrids,
+ymin=0.354813389233575, ymax=2818382931.26445,
+yminorgrids,
+ymode=log,
+ytick style={color=black}
+]
+\addplot [ultra thin, blue]
+table {%
+0 1
+1 10
+2 100
+3 1000
+4 10000
+5 100000
+6 1000000
+7 10000000
+8 100000000
+9 1000000000
+};
+\end{axis}
+
+\end{tikzpicture}

tikzplotlib/line2d.py

@@ -303,9 +303,6 @@ def _table(obj, data):
         xformat = "{}"
         col_sep = ","
         opts = ["header=false", "col sep=comma"]
-        if data["table_row_sep"] != "\n":
-            opts.append("row sep=" + data["table row sep"])
-        content.append("table [{}] {{%\n".format(",".join(opts)))
         data["current axes"].axis_options.append("date coordinates in=x")
         # Replace float xmin/xmax by datetime
         # <https://github.com/matplotlib/matplotlib/issues/13727>.
@@ -322,8 +319,16 @@ def _table(obj, data):
             )
         )
     else:
+        opts = []
         xformat = ff
         col_sep = " "
+
+    if data["table_row_sep"] != "\n":
+        # don't want the \n in the table definition, just in the data (below)
+        opts.append("row sep=" + data["table_row_sep"].strip())
+    if len(opts) > 0:
+        content.append("table [{}] {{%\n".format(",".join(opts)))
+    else:
         content.append("table {%\n")
 
     plot_table = []