#5975: [with patch, positive review] Implement latex output for (combinatorial)
graphs
---------------------------+------------------------------------------------
 Reporter:  rbeezer        |       Owner:  rbeezer                          
     Type:  enhancement    |      Status:  new                              
 Priority:  major          |   Milestone:  sage-feature                     
Component:  graph theory   |    Keywords:                                   
 Reviewer:  John Palmieri  |      Author:  Robert Beezer, Fidel Barrera Cruz
   Merged:                 |  
---------------------------+------------------------------------------------

Comment(by rbeezer):

 Hi Jason,

 The purpose of the line in question is not really a doctest of the
 existence of the packages.

 The function {{{check_tkz_graph()}}} will emit a warning *once* per
 session, if needed, and then be silent.  In order to doctest the latex
 output, I need to be sure that the warning doesn't get mixed in.  So I
 call this function in the doctest, which *may* produce a warning.  Then
 the latex output will be "pure."

 So I need to be certain this line runs, and I don't want to test its
 (variable) output.  If it is tagged "optional" will it always be run in
 testing, no matter what?

 In any event, if there is a change here, I'll make it as part of some
 other revision, so I don't want this to hold up getting this merged.

 Rob

 Replying to [comment:24 jason]:
 > Can I make one small suggestion?
 >
 > This line:
 > {{{
 > check_tkz_graph()  # random - depends on TeX installation
 > }}}
 > ought to be
 > {{{
 > check_tkz_graph()  # optional - depends on the TeX installation
 containing <whatever tex files you need>
 > }}}
 >
 > We really don't like #random doctests if we can help it, and this
 doctest is really testing the existence of an optional program.  I guess
 one argument against this is that it is not testing the installation of an
 optional spkg.
 >
 > Since we have at least one other spkg installing .sty files (sagetex),
 maybe (in a different ticket) we could have a small spkg that installs the
 needed .sty files for this into the sage local tex tree.

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