#13632: Fix latex display of arguments of symbolic functions
-----------------------------+----------------------------------------------
       Reporter:  gouezel    |         Owner:  burcin      
           Type:  defect     |        Status:  needs_review
       Priority:  minor      |     Milestone:  sage-5.5    
      Component:  symbolics  |    Resolution:              
       Keywords:             |   Work issues:              
Report Upstream:  N/A        |     Reviewers:              
        Authors:             |     Merged in:              
   Dependencies:             |      Stopgaps:              
-----------------------------+----------------------------------------------

Comment (by kcrisman):

 Wow, that's a pretty simple fix, but it looks right.  I am a little
 concerned about the following:
 {{{
 sage: type(latex(x))
 <class 'sage.misc.latex.LatexExpr'>
 sage: type(x._latex_())
 <type 'str'>
 }}}
 Will that cause any problem/slow things down, or is this actually an
 improvement?
 {{{
 sage: L = [4,x,3.4]
 sage: timeit("' '.join([z._latex_() for z in L])")
 625 loops, best of 3: 10.1 µs per loop
 sage: timeit("' '.join([latex(z) for z in L])")
 625 loops, best of 3: 17 µs per loop
 }}}
 Naturally fixing the bug is good, just wondering if there is any slick way
 to avoid turning things into !LatexExpr and back to strings - or maybe a
 try/except thing catching the attribute error?

 Otherwise this looks like a good idea and nice fix.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13632#comment:2>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" 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/sage-trac?hl=en.

Reply via email to