On Tue, Mar 2, 2010 at 12:11 AM, Ondrej Certik <ond...@certik.cz> wrote:
> On Tue, Mar 2, 2010 at 12:08 AM, Ondrej Certik <ond...@certik.cz> wrote:
>> Hi,
>>
>> notes for myself:
>>
>> why did you remove the following tests (sympy/core/tests/test_arit.py):
>>
>> - assert list((I*pi).atoms(NumberSymbol)) == [pi]
>> - assert sorted((I*pi).atoms(NumberSymbol, I)) == \
>> - sorted((I*pi).atoms(I,NumberSymbol)) == [pi, I]
>>
>> and:
>>
>> - assert (Rational(1,2)*x).extract_multiplicatively(3) == x/6
>>
>> and
>>
>> - assert (x**(Rational(1,2))).extract_multiplicatively(1/x) ==
>> x**(Rational(3,2))
>>
>>
>> -def test_coeff_expand():
>> - x, y, z = symbols('x y z')
>> - expr = z*(x+y)**2
>> - expr2 = z*(x+y)**2 + z*(2*x + 2*y)**2
>> - assert expr.coeff(z) == 2*x*y + x**2 + y**2
>> - assert expr.coeff(z, expand=False) == (x+y)**2
>> - assert expr2.coeff(z) == 10*x*y + 5*x**2 + 5*y**2
>> - assert expr2.coeff(z, expand=False) == (x+y)**2 + (2*x + 2*y)**2
>>
>>
>> I will try to find this out from your patches, maybe it's explained in
>> the log. Otherwise, it looks good, all tests pass. I need to figure
>> out the above and then also run benchmarks, because some core stuff
>> was modified. Then it's ready to be pushed in.
>
> Oops, I accidentally run tests on the master branch (so I am glad they
> all pass:), but the 1766 branch gives immediately:
>
> $ ./setup.py test
> Traceback (most recent call last):
> File "./setup.py", line 33, in <module>
> import sympy
> File "/home/ondrej/repos/sympy/sympy/__init__.py", line 29, in <module>
> from concrete import *
> File "/home/ondrej/repos/sympy/sympy/concrete/__init__.py", line 2,
> in <module>
> from summations import sum, Sum
> File "/home/ondrej/repos/sympy/sympy/concrete/summations.py", line
> 4, in <module>
> from sympy.solvers import solve
> File "/home/ondrej/repos/sympy/sympy/solvers/__init__.py", line 6, in
> <module>
> from solvers import solve, solve_linear_system, solve_linear_system_LU, \
> File "/home/ondrej/repos/sympy/sympy/solvers/solvers.py", line 34, in
> <module>
> from sympy.polys.polyerrors import *
> ImportError: No module named polyerrors
Chris --- are you also getting this error?
Ondrej
--
You received this message because you are subscribed to the Google Groups
"sympy-patches" group.
To post to this group, send email to sympy-patc...@googlegroups.com.
To unsubscribe from this group, send email to
sympy-patches+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/sympy-patches?hl=en.
