#11641: Implementation of decorator for combinatorial maps
-------------------------------------------+--------------------------------
Reporter: stumpc5 | Owner: sage-combinat
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-pending
Component: combinatorics | Resolution:
Keywords: combinatorial statistic | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Christian Stump | Merged in:
Dependencies: | Stopgaps:
-------------------------------------------+--------------------------------
Comment (by saliola):
Apply: trac_11641-combinatorial_statistics_and_maps_decorator-cs.2.patch,
trac_11641-combinatorial_maps_review-fs.patch,
trac_11641_concrete_combinatorial_maps_cb.patch
Hey Christian,
We made some changes. We deleted combinatorial_statistics since we will do
that in another patch. For now, let's concentrate on combinatorial_maps.
Some changes:
- we changed the __repr__ method
- we added decorators to combinatorial maps (permutations, partitions,
etc.)
- we updated doctests to use the implemented combinatorial maps
But we have some questions:
- I've littered the file combinatorial_maps.py with TODOs. Take a look at
them. Most are documentation issues, or things I don't understand.
- another question: why doesn't the following work?
{{{
sage: from sage.misc.combinatorial_map import
combinatorial_maps_in_class
sage: p = Permutation([1,3,2,4])
sage: cmaps = combinatorial_maps_in_class(p)
sage: cmaps[0](p)
}}}
- should this file be in {{{sage.combinat.misc}}} or
{{{sage.combinat.combinatorial_maps}}} instead of {{{sage.misc}}}?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11641#comment:14>
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.
