#12810: Broken links in the documentation of graph/ files
---------------------------------+------------------------------------------
       Reporter:  ncohen         |         Owner:  jason, ncohen, rlm 
           Type:  defect         |        Status:  needs_review       
       Priority:  major          |     Milestone:  sage-5.1           
      Component:  graph theory   |    Resolution:                     
       Keywords:  sd40.5         |   Work issues:                     
Report Upstream:  N/A            |     Reviewers:  Karl-Dieter Crisman
        Authors:  Nathann Cohen  |     Merged in:                     
   Dependencies:                 |      Stopgaps:                     
---------------------------------+------------------------------------------
Changes (by ncohen):

  * status:  needs_work => needs_review


Comment:

 Hellooooooooooooooooo !!!

 > Couldn't one just change the syntax to say
 > {{{
 > ``sage.graphs.graph_latex.GraphLatex.__graphlatex_options``
 > }}}

 Done !!

 > Also, we are trying to discourage double-underscore attributes.  Such as
 `__graphlatex_options`.   Could this be made single-underscore (perhaps in
 another ticket, if appropriate)?

 Hmmm.. I prefer to let the combinat guys deal with that, it may break code
 on their side.

 > * The doc looks wrong in the (short) tree.pyx - you'll see what I mean.

 Oh, yeah. Columns.

 > * The AUTHORS in matchpoly is indented too much.

 Done, done.

 > * graph_plot.html can have trac #10124 written as {{{:trac:`#10124`}}},
 I think.

 That too.

 Patch updated ! Thank you for the review `:-)`

 Nathann

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