On Thu, May 29, 2014 at 11:42:40AM +0200, Nicolas M. Thiery wrote:
> Here is the description of http://trac.sagemath.org/ticket/16410,
> which potentially needs review:
> 
> ------------------------------------------------------------------------------
> From the discussions of (TODO: add refs to the threads in sage-devel), there 
> is a consensus that, at this point, the combinatorial_map decorator should by 
> default return the decorated method as is, in order to have no impact on 
> speed. On the other hand, projects built on top of Sage, like findstat are 
> welcome to customize locally this hook to instrument the Sage code and 
> exploit the semantic information provided by this decorator.
> 
> This ticket therefore:
> 
>     Defines combinatorial_map as a no-op
>     Discuss the purpose of this decorator
>     Uses the previous implementation as an example of how to customize 
> combinatorial_map 
> 
> Note: this change is slightly backward incompatible since 
> combinatorial_maps_in_class is not functional any more by default.
> ------------------------------------------------------------------------------

Vincent on #16410:
> ...this ticket can be reviewed quickly compared to #16408 (which
> needs some design discussions). We can let this one go and then
> start discussing #16408 but I think we need more opinions on this.

Nathann, Christian, Viviane, ... do you mind briefly stating on the
ticket whether this sounds like a reasonable step forward?

Cheers,
                                Nicolas
--
Nicolas M. ThiƩry "Isil" <nthi...@users.sf.net>
http://Nicolas.Thiery.name/

-- 
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 http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to