#6179: [with patch, needs review] html -- doctest failure in sage-4.0.1.alpha0
---------------------+------------------------------------------------------
Reporter: was | Owner: mhansen
Type: defect | Status: assigned
Priority: blocker | Milestone: sage-4.0.1
Component: doctest | Keywords:
---------------------+------------------------------------------------------
Comment(by jhpalmieri):
I don't care about the efficiency, but I like that my patch is less
ambiguous: with yours, if some unexpected type returns True for
{{{isinstance(blah, int)}}}, then the latex could get screwed up. It also
depends on the ordering in the table. My change was already in a patch
("needs work") for #6089, and in that patch it also gets used when
constructing jsmath expressions. So if you don't have a strong
preference, I'll post my version of the patch here.
This includes your changes to html.py and your new doctest in latex.py,
which I give a positive review to.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6179#comment:6>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
