#5925: [with patch, positive review] Improve speed of
CombinatorialAlgebra.multiply()
-----------------------------+----------------------------------------------
Reporter: jdc | Owner: mhansen
Type: enhancement | Status: closed
Priority: minor | Milestone: sage-4.0.1
Component: combinatorics | Resolution: fixed
Keywords: | Reviewer: Mike Hansen
Author: Dan Christensen | Merged: 4.0.1.rc1
-----------------------------+----------------------------------------------
Comment(by jdc):
Replying to [comment:7 mvngu]:
> Replying to [comment:1 jdc]:
> > Timing test on 2.2GHz Core2Duo running 32-bit Ubuntu 8.04:
{{{
Y=[[1,2,3,4],[5,6]]
time eY=e(Y)
time eY2=eY*eY
}}}
> Hi Dan. I'm showcasing the feature of this ticket in the release tour.
The above code doesn't look sensible to me, since {{{time eY=e(Y)}}}
results in a {{{ValueError}}} under Sage 4.0 and 4.0.1. Can you post
sample code to illustrate the timing improvement introduced by the patch?
The code should illustrate this both before and after applying the patch.
Did you try doing:
{{{
from sage.combinat.symmetric_group_algebra import e
}}}
before running the above code? I don't have sage 4.0 or higher installed
yet, and I'm about to be offline for about a month, so I'm not going to be
able to look into this further for a while.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5925#comment:8>
Sage <http://sagemath.org/>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" 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-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
