[PATCH] Fix for Issue 1920: Now SymTuple.func(*SymTuple.args) work as expected.

2010-04-29 Thread Øyvind Jensen
Added a test and fixed some related whitespace errors. --- sympy/physics/secondquant.py| 20 +--- sympy/physics/tests/test_secondquant.py |9 + 2 files changed, 10 insertions(+), 19 deletions(-) diff --git a/sympy/physics/secondquant.py

Re: [PATCH 1/2] fix matrix test, remove unused imports

2010-04-29 Thread Ondrej Certik
+1 On Tue, Apr 27, 2010 at 2:32 PM, Vinzent Steinberg vinzent.steinb...@googlemail.com wrote: Signed-off-by: Vinzent Steinberg vinzent.steinb...@gmail.com ---  sympy/matrices/tests/test_matrices.py |    6 ++  1 files changed, 2 insertions(+), 4 deletions(-) diff --git

Re: [PATCH 2/2] add a test for wrong args to Matrix()

2010-04-29 Thread Ondrej Certik
+1 On Tue, Apr 27, 2010 at 2:32 PM, Vinzent Steinberg vinzent.steinb...@googlemail.com wrote: Signed-off-by: Vinzent Steinberg vinzent.steinb...@gmail.com ---  sympy/matrices/tests/test_matrices.py |    1 +  1 files changed, 1 insertions(+), 0 deletions(-) diff --git

Re: [PATCH] Fix for Issue 1920: Now SymTuple.func(*SymTuple.args) work as expected.

2010-04-29 Thread Ondrej Certik
Well, I am not sure that the change from: -assert st == SymTuple((1,2,3,4)) to: +assert st == SymTuple(1, 2, 3, 4) is a good idea. In fact I think it's a bad idea, as we migrated from Matrix(1,2, 3, 4) syntax to Matrix([1, 2, 3, 4]). Also imho you want to be able to do

Re: review of Chris' 1766

2010-04-29 Thread Aaron S. Meurer
The failures are 64-bit only: 32-bit: ___ sympy/utilities/tests/test_code_quality.py:test_implicit_imports ___ File