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

Fixes a bug in mutation_class (cluster seed) #14638

Closed
sagetrac-gmoose05 mannequin opened this issue May 24, 2013 · 4 comments
Closed

Fixes a bug in mutation_class (cluster seed) #14638

sagetrac-gmoose05 mannequin opened this issue May 24, 2013 · 4 comments
Assignees
@sagetrac-sage-combinat
Labels
c: combinatorics p: major / 3 t: bug
Milestone
sage-5.10

Comments

@sagetrac-gmoose05
Copy link
Mannequin

sagetrac-gmoose05 mannequin commented May 24, 2013

Reported by Bruce Fontaine: The method mutation_class() was giving the wrong answer for ClusterSeed(['E',8]]).

The length was supposed to be 25080, but our code gives 25067 instead.

The issue was that we refer to index of an element of a list, but we then turn that list into a set.

CC: bfontain@math.cornell.edu @stumpc5

Component: combinatorics

Keywords: cluster mutation class

Author: Gregg Musiker

Reviewer: Christian Stump

Merged: sage-5.10.rc1

Issue created by migration from https://trac.sagemath.org/ticket/14638
@sagetrac-gmoose05 sagetrac-gmoose05 mannequin added this to the sage-5.10 milestone May 24, 2013
@sagetrac-gmoose05
Copy link
Mannequin Author

sagetrac-gmoose05 mannequin commented May 24, 2013

Attachment: trac_14638-mutation_class_fix.patch.gz

@stumpc5
Copy link
Contributor

stumpc5 commented May 24, 2013

Changed keywords from none to cluster mutation class

@jdemeyer
Copy link

jdemeyer commented Jun 3, 2013

Author: Gregg Musiker

@jdemeyer jdemeyer modified the milestones: sage-5.10, sage-5.11 Jun 3, 2013
@jdemeyer
Copy link

jdemeyer commented Jun 5, 2013

Merged: sage-5.10.rc1

@jdemeyer jdemeyer closed this as completed Jun 5, 2013
@sagetrac-gmoose05 sagetrac-gmoose05 mannequin mentioned this issue Jul 3, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-sage-combinat sagetrac-sage-combinat

Labels
c: combinatorics p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.10
Development

No branches or pull requests
2 participants
@stumpc5 @jdemeyer