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