#19385: Refactor DiGraph.__init__
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  ncohen                 |       Status:  needs_review
           Type:         |    Milestone:  sage-6.9
  enhancement            |   Resolution:
       Priority:  major  |    Merged in:
      Component:  graph  |    Reviewers:  Travis Scrimshaw
  theory                 |  Work issues:
       Keywords:         |       Commit:
        Authors:         |  99c8601b947b746cfd0cbbca4c836b82b3ac5dde
  Nathann Cohen          |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/ncohen/19385         |
   Dependencies:         |
  #19381                 |
-------------------------+-------------------------------------------------

Comment (by ncohen):

 > I believe explicit is better than implicit. And it induces a very bad
 gut feeling when `c*G` for a graph G returns a weighted graph or a multi-
 graph depending on whether c is integer or rational.

 +1 here. Not only it drives me mad, but the code has to deal with all that
 and decide it (and God it is ugly), and of course this comes at the cost
 of over-analysing the input, which takes time.

 Nathann

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