#6553: [with patch, needs review] fast slicing of sparse matrices
----------------------------+-----------------------------------------------
Reporter: jason | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-4.1.1
Component: linear algebra | Keywords:
Reviewer: | Author: Jason Grout
Merged: |
----------------------------+-----------------------------------------------
Comment(by jason):
Thanks for reviewing these so quickly!
1. Are you on a 64-bit system (maybe you are getting a different random
matrix)? That doctest quite definitely passes for me. I agree that your
answer seems more correct, though.
2. The error is in the sparse matrix indexing function, not in this
function. I've opened up #6569 to address this issue, with a relevant
example. I don't think that should hold back this code.
3. The condition in (2) should be tested in #6569. I've updated the
patch to include a doctest for (b).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6553#comment:4>
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
-~----------~----~----~----~------~----~------~--~---
