#11529: Rooted trees
-------------------------------------+-------------------------------------
Reporter: hivert | Owner: hivert
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.5
Component: combinatorics | Resolution:
Keywords: rooted trees, | Merged in:
Cayley | Reviewers:
Authors: Florent Hivert | Work issues:
Report Upstream: N/A | Commit:
Branch: | c1aca706fd40900e59024f19a4e29fe190fd65f2
public/combinat/11529 | Stopgaps:
Dependencies: #11407 |
-------------------------------------+-------------------------------------
Comment (by chapoton):
Replying to [comment:32 darij]:
> {{{
> + with self.clone() as t:
> + t.append(other)
> + resu = t
> + return resu
> }}}
> Why not remove the "resu = t" line and just return t? I'm wondering
because I'm not sure if my understanding of the clone context manager is
correct.
Yes, I guess one can return t. I am not very sure either to understand the
clone mantra.
> Also, is it OK if I split `LabelledRootedTrees` into an abstract base
class and a concrete class?
Yes, please do as you want, but try to keep things simple enough for me :)
--
Ticket URL: <http://trac.sagemath.org/ticket/11529#comment:33>
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.
