Opened 4 years ago

Closed 4 years ago

#14638 closed defect (fixed)

Fixes a bug in mutation_class (cluster seed)

Reported by: gmoose05 Owned by: sage-combinat
Priority: major Milestone: sage-5.10
Component: combinatorics Keywords: cluster mutation class
Cc: bfontain@…, stumpc5 Merged in: sage-5.10.rc1
Authors: Gregg Musiker Reviewers: Christian Stump
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

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.

Attachments (1)

trac_14638-mutation_class_fix.patch (1.4 KB) - added by gmoose05 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by gmoose05

comment:1 Changed 4 years ago by stumpc5

  • Keywords cluster mutation class added
  • Status changed from new to needs_review

comment:2 Changed 4 years ago by stumpc5

  • Status changed from needs_review to positive_review

comment:3 Changed 4 years ago by jdemeyer

  • Authors set to Gregg Musiker
  • Milestone changed from sage-5.10 to sage-5.11

comment:4 Changed 4 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.10

comment:5 Changed 4 years ago by jdemeyer

  • Merged in set to sage-5.10.rc1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.