Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
When I use sympy.latex with my report generating code,
I handle the wrapping of equations in my code. So, I
need the raw
Updates:
Labels: Milestone-Release0.6.5 NeedsReview
Comment #1 on issue 1490 by ondrej.certik: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
Comment #3 on issue 1490 by Toon.Verstraelen: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Test runs fine and patch looks good: +1
On line 45, it reads:
return "%s" % tex
Why not just
return tex
Should tex not be a string already?
--
You received this
Comment #4 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Thanks for looking at this. Good point about line 45. I think I was
thinking "I am
95% certain its a string, I'll just put '%s' here just in case." But how
would a
Updates:
Labels: -Latex Printing
Comment #5 on issue 1490 by fab...@fseoane.net: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
I am no Latex expert, but I find a bit confusing the inline=False option:
In [2]: latex(x*y, inline=False)
Out[2]: \be
Updates:
Labels: -NeedsReview
Comment #6 on issue 1490 by ondrej.certik: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
It's confusing to me as well. Otherwise the patch is good in general. Ryan,
could you
please address that?
--
You received thi
Comment #7 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
inline=False must be longer than inline=True. inline=True mean the
equation is
within a sentence like $x+y$. inline=False should be the biggest one
because it is
put
Comment #8 on issue 1490 by ondrej.certik: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
What about:
mode = "inline" $x+y$
mode = "equation" \begin{...
mode = "plain" x + y
or something. We definitely need the "x+y" mode, in fact I think it should
be
Comment #9 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Yes, it seems like a variable with three options would be more transparent
to the
user than my inline=None trick. Should we change the name of the kwarg
from inline
t
Comment #10 on issue 1490 by ondrej.certik: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Let's do the following:
* introduce the mode= parameter and fix it to work with it
* if the user specifies the inline parameter, emit a warning that it's
deprecate
Comment #11 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
OK, I will work on that soon.
--
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 adj
Comment #12 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
OK, so I think I have this implemented. It is on my github repo
git://github.com/ryanGT/sympy.git in the branch latex_mode.
'mode' has 4 options: 'plain', 'inline', 'eq
Comment #14 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
OK, I think I have this fixed and the corresponding changes made to the
tests. The
result is now in my latex_mode branch.
--
You received this message because you are
Comment #15 on issue 1490 by Toon.Verstraelen: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
I'd go for option 2, if there are (other) tests for all possible modes.
--
You received this message because you are listed in the owner
or CC fields of this is
Comment #16 on issue 1490 by Toon.Verstraelen: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
All tests pass, also after a 'git rebase master'.
Could you update the tutorial with the attached patch? (For some reason the
documentation is no longer tested
Comment #17 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Thanks. Ondrej directed me to option 2 as well, so that is what we did.
For some
reason, I can't download that patch. Are you talking about the 10 or so
lines unde
Comment #19 on issue 1490 by ryanlists: LatexPrinter doprint inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
OK, the tutorial has been updated and
python setup.py test_doc
passes.
--
You received this message because you are listed in the owner
or CC fields of this issue, or be
Comment #18 on issue 1490 by Toon.Verstraelen: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
This works at my place:
wget
'http://sympy.googlecode.com/issues/attachment?aid=1757342635748407151&name=0001-Tutorial-update-for-latex-mode-argument.patch.gz'
Updates:
Labels: NeedsReview
Comment #20 on issue 1490 by Vinzent.Steinberg: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of
Updates:
Status: Started
Cc: jorn.baayen
Labels: -NeedsReview NeedsBetterPatch
Comment #21 on issue 1490 by Vinzent.Steinberg: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
The patch looks fine, however there are conflicts with cur
Updates:
Labels: -NeedsBetterPatch NeedsReview
Comment #22 on issue 1490 by Toon.Verstraelen: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Rebased version can be found here:
git://github.com/tovrstra/sympy.git
in the branch 1490 or simply bro
Updates:
Labels: -NeedsReview NeedsBetterPatch
Comment #23 on issue 1490 by Vinzent.Steinberg: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Thanks!
Instead of printing the deprecation warning, Python's native warnings
should be used.
The r
Updates:
Labels: -NeedsBetterPatch NeedsReview
Comment #24 on issue 1490 by Toon.Verstraelen: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Thanks for the advice. It is patched, rebased, tested and available in the
same
branch on github.
--
Updates:
Status: Fixed
Labels: -NeedsReview
Comment #25 on issue 1490 by Vinzent.Steinberg: LatexPrinter doprint
inline=None
http://code.google.com/p/sympy/issues/detail?id=1490
Thank you, this is now merged, finally.
--
You received this message because you are listed in the
24 matches
Mail list logo
