#14542: Implement arithmetic product of cycle index series
----------------------------------------+------------------------------
Reporter: agd | Owner: sage-combinat
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.11
Component: combinatorics | Resolution:
Keywords: species, cycle index | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: u/agd/cis_arith_prod | Dependencies:
Stopgaps: |
----------------------------------------+------------------------------
Comment (by darij):
I've attached a review patch. Sorry for it taking that long; I was waiting
for a thorough explanation of {{{sum_generator}}}, which now has been
added to the otherwise unrelated ticket #13433. The explanation shows that
it is *not* necessary to pad a sequence with zeroes to use
{{{sum_generators}}} on it; {{{sum_generator([a,b])}}} does precisely the
same as {{{sum_generator([a,b,0])}}}. But this isn't really relevant to
this ticket here. What is correct that {{{L([a,b])}}} is *not* the same as
{{{L([a,b,0])}}}, and the latter syntax is the correct one in most
realistic cases. This has nothing to do with {{{sum_generator}}}. Sorry
for the confusion.
I have added some more info to the docstring and the comments, replaced
the arXiv link by the proper syntax for arXiv identifiers (hopefully
correctly), improved PEP 8 compliance (it's {{{def f(n)}}}, not {{{def f
(n)}}}), and replaced {{{n/d}}} for {{{n//d}}} (both do the same when d
divides n, but the latter is faster). If you agree with these all, please
mark this ticket as positive_review.
For the patchbot:
Apply trac_14542_cycle_index_arithmetic_product.patch trac_14542-review-
dg.patch
--
Ticket URL: <http://trac.sagemath.org/ticket/14542#comment:9>
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/groups/opt_out.
