#18032: Broken copy, plot in sandpile
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:
           Type:  defect             |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.6
      Component:  graph theory       |   Resolution:
       Keywords:  sd66               |    Merged in:
        Authors:  Peter Wicks        |    Reviewers:  Nathann Cohen
  Stringfield, Volker Braun          |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  c500bfa13492c6c51ea52a5b9c0b91fe7b47806a
  u/vbraun/ticket/18032              |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by vbraun):

 Is `G.copy` implementing deep copy or shallow copy? The answer is clearly
 "neither" (or, more precisely, deep copy plus a grab bag of stuff that is
 not copy). So it should be labeled more accurately. Then people that
 subclass !GenericGraph wouldn't be so confused by it, this ticket is plain
 evidence that it is confusing. I overrode `__copy__` which now means that
 the grab bag of non-copy stuff is gone from Sandpile. I say good riddance.
 But it does make it inconsistent with what is going on in the base class.

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