Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 1472 by goodrich.ben: [patch] Matrices: Introduce Matrix.vec()
and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
This patch adds two new functions of matrices (and three tests)
.vec() stacks all the col
Updates:
Status: Accepted
Labels: -Type-Defect Type-Enhancement NeedsReview
Comment #1 on issue 1472 by goriccardo: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
Hello, thanks for the patch.
I'm +1 for vech, whi
Comment #2 on issue 1472 by goodrich.ben: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
Thank you. I have done that now in this patch, which needs to be applied
after the
first one.
Attachments:
0001-Matrices-make-vec
Updates:
Labels: Milestone-Release0.6.5
Comment #3 on issue 1472 by goriccardo: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
Great! +1
--
You received this message because you are listed in the owner
or CC fields of t
Updates:
Status: PassedReview
Comment #4 on issue 1472 by Vinzent.Steinberg: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
(No comment was entered for this change.)
--
You received this message because you are listed in
Updates:
Status: Fixed
Comment #5 on issue 1472 by ondrej.certik: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
thanks for the patch. I squashed them to one, and changed "from sympy
import *" to
"from sympy import Matr
Comment #6 on issue 1472 by goodrich.ben: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
In light of this thread
http://groups.google.com/group/sympy/browse_thread/thread/c1059c849c663009
in this patch, I changed the symmetry
Updates:
Status: Started
Labels: -Milestone-Release0.6.5
Comment #7 on issue 1472 by asmeurer: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
(No comment was entered for this change.)
--
You received this messag
Comment #8 on issue 1472 by ondrej.certik: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
Maybe add a kwargument to turn this check off? I can imagine a large
matrix, where it
takes time (or even fails!) to execute simplify o
Updates:
Labels: -NeedsReview NeedsBetterPatch EasyToFix
Comment #9 on issue 1472 by Vinzent.Steinberg: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
Definitely, especially because simplify is not reliable. I would add
Comment #10 on issue 1472 by goodrich.ben: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
OK, this patch is a replacement for the previous one and adds a
checkSymmetry
argument. Although ./bin/test matrices passes all matrix-r
Updates:
Labels: -NeedsBetterPatch NeedsReview
Comment #11 on issue 1472 by asmeurer: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
(No comment was entered for this change.)
--
You received this message because you are
Updates:
Labels: -EasyToFix
Comment #12 on issue 1472 by Toon.Verstraelen: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
Patch is good except for some formatting details and other minor things. See
http://www.python.org
Comment #13 on issue 1472 by goodrich.ben: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
Thanks for the links. I am still just starting to learn Python. I am fine
with your
modified patch. We just need someone else to review
Updates:
Status: Fixed
Comment #14 on issue 1472 by Vinzent.Steinberg: [patch] Matrices: Introduce
Matrix.vec() and Matrix.vech()
http://code.google.com/p/sympy/issues/detail?id=1472
We just need someone else to review it.
No, our rule is that a patch has to be reviewed by at least
15 matches
Mail list logo
