Hi again Dan,

I just browsed through the patch. That definitely sounds good!

On Sun, Sep 05, 2010 at 09:58:01PM +0200, Nicolas M. Thiery wrote:
> I just had a very quick look, and it sounds like what is cached is
> fairly orthogonal to free module operations. So one should be able to
> use a similar caching even with CombinatorialFreeModule. I'll
> investigate this when I'll review.

Apparently the new code is already doing this caching, right?

A quick question: could one use the _from_dict method predefined in
CombinatorialFreeModule instead of the new element_from_dict? This
could possibly have an influence on the speed, since _from_dict uses
its knowledge of the internal data structure to do as little as
possible extraneous work.

Cheers,
                                Nicolas
--
Nicolas M. ThiĆ©ry "Isil" <[email protected]>
http://Nicolas.Thiery.name/

-- 
You received this message because you are subscribed to the Google Groups 
"sage-combinat-devel" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-combinat-devel?hl=en.

Reply via email to