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

fix bug in PermutationGroupElement #2582

Closed
rlmill mannequin opened this issue Mar 18, 2008 · 3 comments
Closed

fix bug in PermutationGroupElement #2582

rlmill mannequin opened this issue Mar 18, 2008 · 3 comments
Assignees
@robertwb
Labels
c: group theory p: major / 3 t: bug
Milestone
sage-2.11

Comments

@rlmill
Copy link
Mannequin

rlmill mannequin commented Mar 18, 2008 

sage: PermutationGroupElement([1,2,4,3,5])
---------------------------------------------------------------------------
<type 'exceptions.AssertionError'>        Traceback (most recent call last)

/Users/rlmill/sage-2.10.4/<ipython console> in <module>()

/Users/rlmill/sage-2.10.4/permgroup_element.pyx in sage.groups.perm_gps.permgroup_element.PermutationGroupElement.__init__()

<type 'exceptions.AssertionError'>:

Component: group theory

Issue created by migration from https://trac.sagemath.org/ticket/2582
@rlmill rlmill mannequin added this to the sage-2.11 milestone Mar 18, 2008
@rlmill rlmill mannequin added c: group theory labels Mar 18, 2008
@rlmill rlmill mannequin assigned robertwb Mar 18, 2008
@rlmill rlmill mannequin added the s: needs review label Mar 18, 2008
@rlmill
Copy link
Mannequin Author

rlmill mannequin commented Mar 18, 2008

Attachment: 2582-perm-gp-elt-list.patch.gz

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 18, 2008

comment:2

Merged in Sage 2.11.alpha0.

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Mar 18, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 18, 2008

comment:3

For the record: You posted a GNU patch and not a mercurial patch, so I ended up with credit in the hg log :). I need to start looking at patches before I merge so that this doesn't happen again. Note: you can export mercurial patches from mercurial ques.

Cheers,

Michael

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

@robertwb robertwb

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

No branches or pull requests
1 participant
@robertwb