#5721: [with patch, needs work] speed up irreducible_character_freudenthal
---------------------------+------------------------------------------------
Reporter: mhansen | Owner: mhansen
Type: defect | Status: new
Priority: major | Milestone: sage-3.4.2
Component: combinatorics | Keywords:
---------------------------+------------------------------------------------
Comment(by bump):
Further testing suggests that it is the hash method in class
{{{AmbientSpaceElement}}}
that is responsible for the most impressive speedup. Remove that method
and the
code is still faster than without the patch, but not very much (48 seconds
versus 59
for {{{sage -t weyl_characters.py}}}).
Removing this hash method has the effect of reordering the dictionaries
that underlie WeylCharacterRing and WeightRing elements. This is why Mike
revised
three tests in weyl_characters.py. The change in test results is harmless,
but it
seems to me that it raises a problem.
In view of this message:
http://groups.google.com/group/sage-combinat-
devel/msg/7e52394814b7779e?hl=en
this would seem to imply that the results of the tests could be different
for 64 bit and 32 bit.
Is this a concern?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5721#comment:7>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
