[PATCH] Added optional boolean argument to coeff to control whether .expand() is called

2009-08-08 Thread Luke
Attached is a patch that allows for an optional boolean argument to coeff which will control whether the expression will be expanded or not. The default behavior is to expand everything. The reason this is useful is if you have an expression which is linear in a term that isn't nested, so you

Re: [PATCH] Added optional boolean argument to coeff to control whether .expand() is called

2009-08-08 Thread Luke
Attached is an updated patch which adds a docstring that describes the new optional keyword argument and provides a doctest to demonstrate the new functionality. ~Luke On Sat, Aug 8, 2009 at 9:24 AM, Lukehazelnu...@gmail.com wrote: Attached is a patch that allows for an optional boolean argument

Re: [PATCH] Added optional boolean argument to coeff to control whether .expand() is called

2009-08-08 Thread Ondrej Certik
It's in, thanks! On Sat, Aug 8, 2009 at 10:38 AM, Lukehazelnu...@gmail.com wrote: Attached is an updated patch which adds a docstring that describes the new optional keyword argument and provides a doctest to demonstrate the new functionality. ~Luke On Sat, Aug 8, 2009 at 9:24 AM,

Subject: [PATCH] Removed .expand() in matrix_multiply, added doctests, alternate implementation

2009-08-08 Thread Luke
Attached is a patch that: 1) removes the .expand() call that was being done in matrix_multiply() 2) adds a doctest and improves the docstring 3) provides and alternative implementation that is somewhat clearer, but about 5% slower. ~Luke --~--~-~--~~~---~--~~ You

Re: Subject: [PATCH] Removed .expand() in matrix_multiply, added doctests, alternate implementation

2009-08-08 Thread Ondrej Certik
On Sat, Aug 8, 2009 at 10:32 PM, Lukehazelnu...@gmail.com wrote: Attached is a patch that: 1) removes the .expand() call that was being done in matrix_multiply() 2) adds a doctest and improves the docstring 3) provides and alternative implementation that is somewhat clearer, but about 5%