#18289: Revert the changes that #17821 made to graph/
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:
           Type:  defect             |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.7
      Component:  graph theory       |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Volker Braun       |    Reviewers:  Nathann Cohen
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/vbraun/faster_output_for_large_graphs|  
ea2ccc2685a477311e113274bdb91937f813c076
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by ncohen):

 > The `graph.show()` method does its own thing, though the graph plotting
 (wtf is `GraphPlot`) reinvents the wheel in slightly less round ways so
 often that I have lost count.

 I barely ever touched the file, except to add some doc and handle default
 options somewhat correctly. If you think that we can do without it by
 using some more general code I really have no objection to make.

 About the difference between show/pretty_print/repr, I have to agree with
 Karl-Dieter that I find the differences rather thin. Maybe `_rich_repr_`
 makes sense for latex output which would look better in the notebook, and
 would be impossible to read in the console? For graphs, however, there is
 no general way to have a 'readable/enlightenning' plot, so the best is
 probably to not represent it graphically unless requested explicitly.

 Nathann

--
Ticket URL: <http://trac.sagemath.org/ticket/18289#comment:13>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to