#11641: Implementation of decorator for combinatorial maps
-------------------------------------------+--------------------------------
Reporter: stumpc5 | Owner: sage-combinat
Type: enhancement | Status: needs_info
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 stumpc5):
Replying to [comment:14 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
Should I delete all patches on the ticket, and you again add those we want
to keep?
> 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.
Alright.
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
good - but I guess we are creating tons of conflicts in the queue if
trac_11641_concrete_combinatorial_maps_cb.patch gets merged into main sage
(though I guess we should do it anyway).
> 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) }}}
We want to call this function as in
{{{
sage: p.inverse()
}}}
so
{{{
sage: cmaps[0]()
}}}
returns p.whatevercmapsof0is(). Does that make sense to you?
- should this file be in `sage.combinat.misc` or
`sage.combinat.combinatorial_maps` instead of `sage.misc` ?
Adding it to `sage.combinat.misc` or `sage.combinat.combinatorial_maps`
sounds good to me; I basically copied the behaviour of some other
decorator that happend to be in `sage.misc`.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11641#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 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.
