#13425: Compute mutation type of a ClusterSeed or ClusterQuiver
---------------------------------------------------+------------------------
Reporter: gmoose05 | Owner:
sage-combinat
Type: enhancement | Status:
needs_review
Priority: major | Milestone: sage-5.8
Component: combinatorics | Resolution:
Keywords: cluster algebra, quiver, days45 | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Gregg Musiker, Christian Stump | Merged in:
Dependencies: #13424 | Stopgaps:
---------------------------------------------------+------------------------
Comment (by gmoose05):
Christian, I just posted a new review patch. I think the ticket is not
too far away from being ready. I documented as much of the code as I
could with comments, and added explanations and examples for almost all
methods.
There are a few outstanding comments or questions marked with CHECK.
Also, I see that in cluster_seed.py we add mutation_type() commands every
time the user asks for a Cluster Variable or related command. However,
this greatly slows down those methods with the only advantage being the
almost_positive_roots command. So I figured we should discuss this over
email or skype at some point.
We also discussed at Sage Days about trying to optimize by having the
program test for infinite_mutation_type by randomly mutating a few times
if more than 6 vertices before trying to test for a recognizable
mutation_type. I haven't made such changes.
Gregg
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13425#comment:9>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
