#6089: [with patch, needs review] view command: don't always use jsMath
-------------------------+--------------------------------------------------
 Reporter:  jhpalmieri   |       Owner:  jhpalmieri
     Type:  enhancement  |      Status:  new       
 Priority:  minor        |   Milestone:  sage-4.0.2
Component:  misc         |    Keywords:            
 Reviewer:               |      Author:            
   Merged:               |  
-------------------------+--------------------------------------------------

Comment(by jhpalmieri):

 Here's a new patch: avoid-jsmath-version4.patch.  I've also attached
 avoid-delta34.patch, which is the difference between version3 and
 version4, in case that makes refereeing any easier.

 A summary of the changes:

  - I've added doctests to many of the functions in latex.py, so we're over
 90% coverage now.  (For some reason, {{{sage -coverage}}} wasn't seeing
 the doctest for {{{pretty_print}}}, which is why I added "# indirect
 doctest" there.)

  - I made some of the reST cleaner and fancier (with cross-references, as
 described in #6226).

  - I reworked {{{_run_latex_}}} (which used to be called
 {{{_run_latex_on_file_}}}, which I decided was sort of redundant).  For me
 at least, this seems to behave right with the various combinations of the
 presence/absence of dvipng and convert.  I also added some comments so
 that the structure would be clearer. (I'm not sure what else you mean
 about it being hard to follow.) I think that running in the background
 might be problematic, but this is off by default anyway.

 As far as the {{{view([k,g])}}} issue, that's purely a LaTeX thing: the
 code
 {{{
 \[  <code for k>, <code for g> \]
 }}}
 gets passed directly to LaTeX, and that's why the vertical spacing and the
 size of the brackets is the way it is.  Maybe we could modify the LaTeX
 code for either k or g to fix this, but since these are toy examples
 anyway, I don't think it's worth fixing.  My suggestion would instead be
 to either not view graphs or knots as members of lists, or write the
 {{{_latex_}}} methods for the real Sage objects so that they work well in
 this context.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6089#comment:17>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to