Implementation of the class ClusterSeed

[stumpc5]

This class implements cluster seeds for skew-symmetrizable matrices.

The patch contains multiple examples.

sage: S = ClusterSeed(['A',3]); S
A seed for a cluster algebra of rank 3 of type ['A', 3]

Depends on #10538

Component: combinatorics

Keywords: cluster algebra, quiver, days45

Author: Christian Stump, Gregg Musiker

Reviewer: Salvatore Stella

Merged: sage-5.8.beta0

Issue created by migration from https://trac.sagemath.org/ticket/13369

[sagetrac-gmoose05]

comment:2

• trac_13369-cluster_seed-cs-NEW.patch is an exact copy of trac_13369-cluster_seed-cs.patch, except in quiver.py where the change to lines 97-100 now apply to lines 201-204 instead. (Also one typo fixed.)

This was done because trac_13369-cluster_seed-cs.patch no longer applies after recent changes to 10538, although it really was only this four line conflict.

• More substantially, trac_13369-cluster-seed-gm.patch fixes compatibility issues with recent changes to 10538 and 10527. One can now apply all patches in 10527 in order, then all patches in 10538 in order, then the two patches

trac_13369-cluster_seed-cs-NEW.patch followed by trac_13369-cluster_seed-gm.patch,

and then the rebuilt sage passes all doc tests and appears to work correctly.

• A more substantial review will be forthcoming.

[stumpc5]

comment:5

Replying to @sagetrac-gmoose05:

I just renamed and uploaded the two relevant patches without any changes.

[sagetrac-gmoose05]

comment:6

Also, despite the patchbot's inability to apply the patches, it works fine with 5.3.rc1 on my machine (with the 10527 and 10538 patches applied first).

[stumpc5]

comment:7

Both patches rebased according to the new lazy import. No other changes.

[Etn40ff]

Reviewer: Salvatore Stella

[stumpc5]

comment:9

Hi,

I updated my first patch (and uploaded Gregg's patch without changes). This resulted in some missing doctests which I then added in yet another patch. We also had some old things in the ClusterVariable class which I removed. If we need that back later, we can do it in another ticket...

Cheers, Christian

[stumpc5]

Changed author from Christian Stump to Christian Stump, Gregg Musiker

[stumpc5]

Attachment: trac_13369_cluster_seed.patch.gz

[stumpc5]

Attachment: trac_13369-review.patch.gz

[stumpc5]

comment:12

Attachment: trac_13369-second_review.patch.gz

[Etn40ff]

Changed keywords from cluster algebra, quiver to cluster algebra, quiver, days45

[jdemeyer]

Merged: sage-5.8.beta0