Agreed. In fact, it seems to me the proper syntax is
sage: p = gap(PermutationGroupElement('(1,2,3)'))
sage: q = gap(PermutationGroupElement('()'))
sage: gap.Group([p, q])
Group( [ (1,2,3), () ] )
sage: gap.Group([p]) == gap.Group([p, q])
True
and here it all works fine. I think Permutation (which is part of
Mike Hansen's combinat code) happens to play well with
PermutationGroupElement but it is really designed for something else.
On Sat, Nov 1, 2008 at 9:00 AM, mabshoff <[EMAIL PROTECTED]> wrote:
>
>
>
> On Nov 1, 5:37 am, "David Joyner" <[EMAIL PROTECTED]> wrote:
>> On the other hand,
>>
>> sage: gap.Group(["(1,2)","()"])
>> Group( [ (1,2), () ] )
>> sage: gap.Group(["(1,2)"])
>> Group( [ (1,2) ] )
>> sage: gap.Group(["(1,2)","()"]) == gap.Group(["(1,2)"])
>> True
>>
>> so GAP does allow the identity as a member of the list of generators.
>
> Ok, looking at the problem again: Since you are passing arguments
> directly to GAP without Sage checking anything this is probably a user
> error, hence no ticket.
>
> Cheers,
>
> Michael
> >
>
--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/sage-devel
URLs: http://www.sagemath.org
-~----------~----~----~----~------~----~------~--~---
