Hi Florent and Nicolas, On 13 Jun., 13:29, "Nicolas M. Thiery" <[email protected]> wrote: > On Mon, Jun 13, 2011 at 01:03:07PM +0200, Florent hivert wrote: > > So I'd like to have a vote for either one of those four options: > > > 1. unimport BinaryTree from sage.misc.sage_ds and import them from combinat > > > 2. leave think as such. Find a different name for the mathematical binary > > trees (eg: BinaryTreesCombinatorial). > > > 3. rename BinaryTree from sage.misc.sage_ds (eg: BinaryTreesDataStructure) > > and import BinaryTree from combinat > > My vote: 1 or 3. Like most of the sage-combinat people I guess :-)
I need more information: How do the binary trees from sage.misc.sage_ds and from combinat compare, performance-wise? In particular, will there be a performance regression for the "compiled polynomials"? If there is a regression then the combinat binary trees should learn from sage.misc.sage_ds. A backward-incompatible change is out of question if it is combined with a regression, IMHO. If there is no regression and if compiled polynomials really is the only application of sage.misc.sage_ds, then I wouldn't complain about a change. Cheers, Simon -- To post to this group, send an email to [email protected] To unsubscribe from this group, send an email to [email protected] For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
