[sage-combinat-devel] Re: reject in categories-root_system-6588.patch

2011-02-26 Thread Christian Stump
Hi Nicolas, > What's your time line for #10817? I would like to get #6588 fairly > soon into Sage (sorry, it did not show up in the current organization > of the queue). So there is a question of which should go first. The associahedron works quite fine now, but is still lacking some functionalit

[sage-combinat-devel] sage combinat install failed

2011-02-26 Thread Christian Stump
Hi, I got a mail from someone who wanted to install sage and the combinat queue in order to look at the cluster algebra package. Here is the error code he got, maybe someone knows how to handle it, I don't: > $ ./sage -combinat install > Creating sage-combinat branch: >  /Applications/sage/sage

Re: [sage-combinat-devel] #7922 timing issues

2011-02-26 Thread Nicolas M. Thiery
On Fri, Feb 25, 2011 at 06:12:15PM +0100, Nicolas M. Thiery wrote: > - What would be an efficient implementation of hashing for free >module elements? The one you implemented in AmbientSpace can >be 10 times faster than the default one provided by SageObject and >using repr. So it wo

Re: [sage-combinat-devel] reject in categories-root_system-6588.patch

2011-02-26 Thread Nicolas M. Thiery
Hi Christian! On Mon, Feb 21, 2011 at 07:32:47AM -0800, Christian Stump wrote: > I added some methods in root_lattice_realization in order to construct > the generalized associahedron. This cause the patch in the title to > get some rejects, I added another patch right after containing tho

Re: [sage-combinat-devel] #7922 timing issues

2011-02-26 Thread Nicolas M. Thiery
Hi Dan, I have just been through your #7922 patches, and put a reviewers patch in the Sage-Combinat queue: trac_7922-review-nt.patch Altogether it looks good. I did a few changes, and also added a series of comments and suggestions. Please check them out, and decide which ones yo