#16741: Bug in IncidenceStructure.relabel when the ground set consists of pairs
-----------------------------------------+----------------------------
Reporter: ncohen | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: combinatorial designs | Resolution:
Keywords: | Merged in:
Authors: Nathann Cohen | Reviewers:
Report Upstream: N/A | Work issues:
Branch: u/ncohen/16741 | Commit:
Dependencies: | Stopgaps:
-----------------------------------------+----------------------------
Changes (by ncohen):
* status: new => needs_review
* branch: => u/ncohen/16741
Old description:
> Before
>
> {{{
>
> }}}
>
> After
>
> {{{
>
> }}}
New description:
Before
{{{
sage: designs.IncidenceStructure([[(1,2),(3,4)]]).automorphism_group()
...
ValueError: Invalid permutation vector: [(3, 4), (1, 2)]
}}}
After
{{{
sage: designs.IncidenceStructure([[(1,2),(3,4)]]).automorphism_group()
Permutation Group with generators [((1,2),(3,4))]
}}}
--
Comment:
This really was triky to fix. The permutation code tries to handle several
incompatible input i.e. tries to guess whether input is given in list
notation or cycle notation, and guesses that from the type list/tuple of
input.
Plus some permutation code WANTS permutations on 1...n, others on
0,...,n-1 and others handle everything...
--
Ticket URL: <http://trac.sagemath.org/ticket/16741#comment:1>
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/d/optout.
