Hey Salvatore, The typical approach is to put a stopgap, but it is best to do it in such a way that is as minimal as possible (i.e., hopefully not every ClusterQuiver from a matrix will have this stopgap pop up, or at worst only when the input is a matrix).
Best, Travis On Monday, February 27, 2017 at 10:02:50 AM UTC-6, Salvatore Stella wrote: > > Dear all, > some time ago I reported a bug in the implementation of `ClusterQuiver` > which > yields wrong answers. The relevant ticket is #22381. To give you an idea > things like this happen: > > {{{ > sage: B = Matrix([[0,1,0],[-1,0,1],[0,-1,0],[2,0,0]]) > sage: Q = ClusterQuiver(B) > sage: Q.is_mutation_finite() # This should return True > False > }}} > > After some thought I realized that properly fixing all the issues in that > ticket will require quite a lot of work: they all stem from how the data > structure is implemented. Unfortunately I am currently unable to set aside > enough time to handle it. Which is the best way of proceeding here? > Should I > add a warning message and leave it be? Is there any other option > available? > Thanks > S. > > > -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at https://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/d/optout.