#11386: Add a bracelet combinatorial class
------------------------------+---------------------------------------------
Reporter: DanielRecoskie | Owner: nthiery
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.0
Component: combinatorics | Keywords: sage-combinat combinatorics
bracelet
Work_issues: | Upstream: N/A
Reviewer: | Author: Daniel Recoskie
Merged: | Dependencies:
------------------------------+---------------------------------------------
Comment(by nthiery):
Replying to [comment:7 ncohen]:
> Well, I guess it is possible but is it worth doing ? I expect the
"common code" would be much harder to understand (with "if" everywhere
slowing it down :-D) than two dedicated ones, and having two versions of a
20-lines code is not that bad an issue anyway... What do you think ?
It's just a suggestion, I let the author decide depending on how close the
algorithms are. In the case of Lyndon words / necklaces, the algorithm
could be "parametrized" to handle both cases at once. When natural, this
is nice from a duplication point of view, but also for "Sage as a library
of algorithm": the reader sees right away that there is really a single
algorithm.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11386#comment:8>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
