#4502: [with patch; needs work] numerical noise in matrix_double_dense on intel
mac os X 10.5: inverting a singular matrix
-------------------------------------+--------------------------------------
Reporter: jhpalmieri | Owner: somebody
Type: defect | Status: new
Priority: major | Milestone: sage-3.2
Component: linear algebra | Resolution:
Keywords: numerical noise, matrix |
-------------------------------------+--------------------------------------
Comment (by GeorgSWeber):
Hi John,
>Could the error message
{{{
raise ValueError, "self must be square"
}}}
>be changed to something like "matrix must be square"? Actually, why is
this here? I get an error message about non-square matrices (I think
produced by scipy) before applying the patch.
This test and error message were copied verbatim from lines 759/760 of
that same file (function "def determinant(self)"). In my first patch, I
introduced the additional calculation of the determinant in the function
for inverting the matrix, and I wanted to make sure that all necessary
sanity checks were in place.
Would it be possible for you to provide not only a concrete wording, but a
new trac ticket with patch, if you like some other wording better for that
error message? I'll review it then. (Maybe just removing this test from
the .pyx file and relying on scipy producing the error message for us also
for determinants on non-square matrices is an option here.)
Cheers,[[BR]]
gsw
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4502#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
-~----------~----~----~----~------~----~------~--~---
