What methods were you thinking of moving from MatrixBase to MutableMatrix? Are these methods also relevant for ImmutableMatrix?
Background: MatrixBase was created when ImmutabileMatrices were created. It is effectively all of the shared code between MutableMatrix and ImmutableMatrix. It could more accurately be called DenseMatrix. It would be reasonable to factor out a large chunk of MatrixBase into something more general that could be more easily shared with SparseMatrix. Perhaps we should do the following: Rename MatrixBase -> DenseMatrix Create new MatrixBase and factor out some agnostic code into it. Do you have a branch or pull request that we should look into? At this point I'm not willing to take on SparseMatrix (work is busy) but I am willing to review. Also, can someone repost why SparseMatrix is important? It is important internally, right? How quickly after creating a good SparseMatrix implementation would SymPy reap these benefits? On Wed, Sep 26, 2012 at 12:27 AM, smichr <[email protected]> wrote: > I've spent the last several days trying to incorporate Sherjilozair's > smallchanges PR into matrices. > > matrices is still rough in design, namely there are lots of methods in > MatrixBase that should probably be in MutableMatrix since SparseMatrix > derives from MatrixBase and Sparse's .mat is a dictionary while Matrix's > .mat is a flat list of row-wise elements. To keep make it clear when a > method shouldn't work with Sparse I've changed Sparse's .mat attribute to > .smat so if a method is called on Sparse that is defined in MatrixBase (but > really should be in MutableMatrix) it will now fail. smat and mat should > probably be private methods. > > I hope lack of the ideal will not keep someone from looking at this and > considering the significant upgrade. I also un-XFAILed the test suite for > SparseMatrix which has been un-run since almost the begninning of SymPy > (about 10,500 commits ago where an immediate return was added). > > If there are any changes that one wants to make, feel free to take over > the branch and make the changes. Just as there is no end to the writing of > books, there is no end to changes that can be made...only the possibility > of trading one work for another :-) > > The coverage is above 92%. > > Best regards, > Christopher Smith > > -- > You received this message because you are subscribed to the Google Groups > "sympy" group. > To view this discussion on the web visit > https://groups.google.com/d/msg/sympy/-/Uiq4i-Oo20cJ. > 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/sympy?hl=en. > -- You received this message because you are subscribed to the Google Groups "sympy" 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/sympy?hl=en.
