#12029: Fast conversion of ClonableIntArray to list
-----------------------------+----------------------------------------------
   Reporter:  SimonKing      |          Owner:  sage-combinat
       Type:  enhancement    |         Status:  needs_review 
   Priority:  major          |      Milestone:  sage-4.8     
  Component:  combinatorics  |       Keywords:               
Work_issues:                 |       Upstream:  N/A          
   Reviewer:                 |         Author:  Simon King   
     Merged:                 |   Dependencies:               
-----------------------------+----------------------------------------------

Comment(by hivert):

 Hi Simon,

 Looking at the C code I figured out that replacing
 {{{
 cdef list L = PyList_New(self._len)
 }}}
 by
 {{{
 cdef list L = <list> PyList_New(self._len)
 }}}
 allows to gain a few more speed:
 {{{
 sage: %timeit I.list()
 625 loops, best of 3: 13 µs per loop
 }}}
 So one less microsecond. Can you fold my patch and review it. I'm Ok with
 your changes.

 Cheers,

 Florent

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12029#comment:12>
Sage <http://www.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.

Reply via email to