Updates:
Status: Started
Comment #6 on issue 1282 by ondrej.certik: missing \times in latex output
http://code.google.com/p/sympy/issues/detail?id=1282
Nice patch, thanks! There is only one problem --- previously, the
mul_symbol kwarg
was not properly tested, so I wrote tests for it (patch attached), and it
works. With
your new patch however, it stops working (you couldn't discover it, since
we didn't
have tests for it). I rebased your patch, so just apply the two attached
patches and
you'll get:
$ bin/test sympy/printing/tests/test_latex.py
============================= test process starts
==============================
executable: /usr/bin/python (2.6.2-final-0)
sympy/printing/tests/test_latex.py[17] ........f......F.
[FAIL]
________________________________________________________________________________
___________ sympy/printing/tests/test_latex.py:test_latex_mul_symbol
___________
File "/home/ondrej/repos/sympy/sympy/printing/tests/test_latex.py", line
158, in
test_latex_mul_symbol
assert latex(4*4**x, mul_symbol='dot') == "$4 \\cdot 4^{x}$"
AssertionError
======== tests finished: 15 passed, 1 failed, 1 xfailed in 0.04 seconds
========
DO *NOT* COMMIT!
It's because it now always returns \\times, even if I tell it to return
\\cdot using
the mul_symbol parameter. So this small problem has to be fixed, then I'll
push it in.
Thanks again for the work.
Attachments:
0001-Tests-for-latex-.-mul_symbol-added.patch 1.1 KB
0002-Latex-_print_Mul-fix-issue-1282.patch 3.3 KB
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sympy-issues?hl=en
-~----------~----~----~----~------~----~------~--~---
