#18032: Broken copy, plot in sandpile
-------------------------------------------------+-------------------------
       Reporter:  vbraun                         |        Owner:
           Type:  defect                         |       Status:  new
       Priority:  major                          |    Milestone:  sage-6.6
      Component:  graph theory                   |   Resolution:
       Keywords:                                 |    Merged in:
        Authors:                                 |    Reviewers:
Report Upstream:  N/A                            |  Work issues:
         Branch:                                 |       Commit:
  u/peterwicksstringfield/ticket/18032           |     Stopgaps:
   Dependencies:                                 |
-------------------------------------------------+-------------------------
Changes (by peterwicksstringfield):

 * branch:   => u/peterwicksstringfield/ticket/18032


Comment:

 The inherited __copy__ method was broken because Sandpile changed the
 signature of __init__.

 I fixed this by moving the part of __copy__ that depends on __init__'s
 signature into its own function and overriding it in Sandpile.

 This could also be fixed by copy+pasting the entire __copy__ method into
 Sandpile and tweaking it a bit. I think that's uglier than this though,
 though maybe not by much.

 What does everyone else think?

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