On 10/31/12 6:51 AM, Raniere Gaia Silva wrote:
In the matrix's class diagram (see source:sage/matrix/docs.py) for dense
matrix are list the subclass
* Matrix_real_double_dense
* Matrix_complex_double_dense
which already are implemented. For sparse matrix it suggest the creation
of the subclass
* Matrix_RDF_sparse
* Matrix_CDF_sparse
What I like to know are if will be better to replace
* Matrix_RDF_sparse => Matrix_real_double_sparse
* Matrix_CDF_sparse => Matrix_complex_double_dense
Since most algorithms are shared between real and complex, most things
are actually in the superclass Matrix_double_dense
(matrix_double_dense.pyx). I'd suggest the same for sparse matrices.
As for the names, I could see arguments both ways. Probably consistency
with the existing names is a good thing, but on the other hand, I
wouldn't be opposed to changing the dense classes to Matrix_RDF_dense
and Matrix_CDF_dense.
Thanks,
Jason
--
You received this message because you are subscribed to the Google Groups
"sage-devel" group.
To post to this group, send email to sage-devel@googlegroups.com.
To unsubscribe from this group, send email to
sage-devel+unsubscr...@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel?hl=en.
