id,summary,reporter,owner,description,type,status,priority,milestone,component,resolution,keywords,cc,merged,author,reviewer,upstream,work_issues,branch,commit,dependencies,stopgaps
22765,Fix cycle_index for permutation groups on an arbitrary domain + cleanup,nthiery,,"The code for `cycle_index` predates permutation groups with arbitrary
domain, as well as their `cycle_type` methods. Instead it goes through
plain Permutation's, which only works if the domain is 1,...n:
{{{
sage: G = PermutationGroup([['b','c','a']], domain=['a','b','c'])
sage: G.cycle_index()
ValueError Traceback (most recent call last)
/opt/sage-git2/local/lib/python2.7/site-packages/sage/combinat/permutation.pyc in __init__(self, parent, l, check_input)
519 for i in lst:
520 try:
--> 521 i = int(i)
522 except TypeError:
523 raise ValueError(""the elements must be integer variables"")
ValueError: invalid literal for int() with base 10: 'a'
}}}
This ticket fixes that, and uses the occasion to clean a bit the code,
using in particular the `conjugacy_classes` method to avoid a direct
access to GAP, and the docstring.
This bug was discovered while updating the book ""Calcul Mathématique avec Sage"".
",defect,closed,major,sage-8.0,group theory,fixed,"gap, permutation group, Pólya enumeration",nborie dimpase,,Nicolas M. Thiéry,Frédéric Chapoton,N/A,,e7add6def823f041ea6f70391b88169aa28088e5,,,