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

bug in the method conjugacy_classes_subgroups #981

Closed
wdjoyner opened this issue Oct 24, 2007 · 2 comments
Closed

bug in the method conjugacy_classes_subgroups #981

wdjoyner opened this issue Oct 24, 2007 · 2 comments
Assignees
@wdjoyner
Labels
c: combinatorics p: minor / 4 t: bug
Milestone
sage-2.8.9

Comments

@wdjoyner
Copy link

there is a bug in the module permgroup.py, at line 1226,
the method conjugacy_classes_subgroups calls PermutationGroupElement;
I think it should be PermutationGroup.
reported by Biel

CC: @sagetrac-sage-combinat

Component: combinatorics

Issue created by migration from https://trac.sagemath.org/ticket/981
@wdjoyner wdjoyner added this to the sage-2.8.9 milestone Oct 24, 2007
@wdjoyner wdjoyner self-assigned this Oct 24, 2007
@wdjoyner
Copy link
Author

comment:1

Patch is here:
http://sage.math.washington.edu/home/wdj/patches/permgp-patch.hg
Passes sage -t. Created agains sage 2.8.3 on a suse amd64 machine,
so may be a bit old....

@malb
Copy link
Member

malb commented Oct 24, 2007

comment:3

applied to 2.8.9.alpha1

@malb malb closed this as completed Oct 24, 2007
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wdjoyner wdjoyner

Labels
c: combinatorics p: minor / 4 t: bug
Projects
None yet
Milestone
sage-2.8.9
Development

No branches or pull requests
2 participants
@wdjoyner @malb