#5721: [with patch, needs review] 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):

 I have factored the patch trac_5721-part, and changed the doctests to be
 deterministic.

 The first patch, trac_5721-a.patch contains *only* docstring revision.
 The output of {{{.mlist()}}} is sorted in three places to make the test
 deterministic.

 The second patch, trac_5721-b.patch contains only code revision. These
 changes are the same as in trac_5721-part.patch, and are a subset of
 Mike's original patch.

 With the docstring changes, the tests pass either before or after the
 second patch. But after the second patch, there is a better than 3-fold
 speed improvement.

 Dan

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5721#comment:10>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to