Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Zero as a symbol for a permutation group #14845

Closed
mwhansen opened this issue Jul 1, 2013 · 5 comments
Closed

Zero as a symbol for a permutation group #14845

mwhansen opened this issue Jul 1, 2013 · 5 comments
Assignees
@wdjoyner
Labels
c: group theory p: major / 3 t: bug
Milestone
sage-5.12

Comments

@mwhansen
Copy link
Contributor

mwhansen commented Jul 1, 2013 

sage: PermutationGroup([(0,1)])
Traceback (most recent call last)
...
ValueError: Invalid permutation vector: [(0, 1)]

We expanded the possibilities for symbol sets, and it would be nice if this 0-index case was picked up properly.

Component: group theory

Author: Mike Hansen

Reviewer: Nathann Cohen

Merged: sage-5.12.beta1

Issue created by migration from https://trac.sagemath.org/ticket/14845
@mwhansen mwhansen added this to the sage-5.12 milestone Jul 1, 2013
@mwhansen

This comment has been minimized.

Sign in to view
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jul 1, 2013

comment:3

Attachment: trac_14845-mh.patch.gz

Good to go !

Nathann

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jul 1, 2013

Reviewer: Nathann Cohen

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Jul 2, 2013

comment:4

Thanks, Mike and Nathann, that was fast! -Rob

@jdemeyer
Copy link

Merged: sage-5.12.beta1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wdjoyner wdjoyner

Labels
c: group theory p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.12
Development

No branches or pull requests
3 participants
@wdjoyner @mwhansen @jdemeyer