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/dev/peps/pep-0008/ and http://www.python.org/dev/peps/pep-0257/
for more info on general python coding style conventions. A test for
check_symmetry=False is missing. I would also raise a ValueError instead of a
TypeError when the symmetry check fails.

A modified patch is attached.

All tests pass on my machine.

Attachments:
        0001-add-check_symmetry-True-argument-to-vech-method.patch  2.9 KB

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" 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-issues?hl=en.

Reply via email to