#13601: Module to load matrix from file
----------------------------------+-----------------------------------------
Reporter: r.gaia.cs | Owner: jason, was
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.5
Component: linear algebra | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: | Merged in:
Dependencies: | Stopgaps:
----------------------------------+-----------------------------------------
Comment (by Bouillaguet):
+1 to use matrix.load() to load the matrix !
This ticket should really be rebased on top of #13678 !
Minor problems I spotted : there are a lot of english mistakes in the
patch, copyright notice should not say 2005 W.stein, but 2012 R.Silva.
Lastly, I find it weird to write a whole lot of boilerplate, especially in
the (user-visible) documentation, about things that are not implemented. I
think it could confuse users, and I personally think that it would be
better if we took the pain of actually doing the work before merging the
ticket. Or at least it could be rephrased "TODO : other formats that we
could possibly handle in the future include...."
About the !MatrixMarket format : as Raniere mentions, !SciPy has functions
to read/write this format. These functions could probably be used as-is
for floating-point matrices, but as far as I understand what they do, they
do not support integer matrices. And even if they would, they would be
limited to 32/64 bits integers because they rely on !NumPy. Therefore I
tend to believe that an integer version of the !MatrixMarket code would
have to be re-implemented. We cannot patch !SciPy to work with arbitrarily
large integers... So I think that we should take !SciPy's !MatrixMarket
code and improve it. I am willing to do this, but I'd rather be sure that
we agree before doing it.
Oh, a last detail : the !MatrixMarket format comes in a dense and in a
sparse version, real, complex, integer or binary, thus it would seem
natural that the loader itself sets the sparsity flag and the ring. In the
patch they are given to the loader.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13601#comment:14>
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.
