#4973: [with patch, needs review] rewrite the function __getitem__ in 
matrix0.pyx
to not explicitly use the python/C api
----------------------+-----------------------------------------------------
 Reporter:  mabshoff  |        Owner:  jason   
     Type:  defect    |       Status:  assigned
 Priority:  major     |    Milestone:  sage-3.3
Component:  misc      |   Resolution:          
 Keywords:            |  
----------------------+-----------------------------------------------------
Comment (by craigcitro):

 New version of the patch. So this adds one additional Python/C API call,
 but gets a 10% speedup for the case of `M[i,j]`, which I think is
 worthwhile. Furthermore, it's only necessary because this is a case where
 Cython doesn't do enough specific optimization for us -- one day, when
 Cython gets smarter, we can clean this up more.

 Also fixed a bug in matrix indexing with tuples, and added a doctest for
 that case.

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