PermutationGroup.cardinality is sometimes an int

[vdelecroix]

sage: P = PermutationGroup(['(1,2)','(1,3)'])
sage: P.cardinality()
6
sage: type(P.cardinality())
<type 'int'>

This prevents using the .is_one() method or any other Integer specific method.

The problem comes from the shortcuts implemented in the _order method.

[vdelecroix]

New commits:

​f441434

23778: fix cardinality for PermutationGroup

[tscrim]

Did you also check to see how

        cycle_tuples = []
        for g in gens:
            temp = g.cycle_tuples()
            if len(temp) != 2:
                return None
            cycle_tuples.append(temp)

compares timing-wise? At least it seems like it would be faster...

[vdelecroix]

I don't believe that cycle_tuples would be the culprit, but your version is cleaner.

[git]

Branch pushed to git repo; I updated commit sha1. New commits:

​1907f8b

23778: simpler loop

[tscrim]

One failing test according to the patchbot:

sage -t --long src/sage/sets/set_from_iterator.py  # 1 doctest failed

[git]

Branch pushed to git repo; I updated commit sha1. New commits:

​e9795a0

23778: fix a doctest in set_from_iterator.py

[vdelecroix]

Indeed... the source code had changed.

[vdelecroix]

Fixed!

[tscrim]

Thanks.