Comment #8 on issue 2110 by Vinzent.Steinberg: failing tests
http://code.google.com/p/sympy/issues/detail?id=2110

It triggers some bug in subs. If you revert a line introduced by

commit 4e5195d303eca6997c77b57144ee45829b70a2ba
Author: Bill Flynn <wfly...@gmail.com>
Date:   Sat Aug 22 11:55:47 2009 -0700

    Fixes Mul's substitution

Lots of tests written. Previously it was essentially broken, now it works both
    for commutative and noncommutative symbols

    Signed-off-by: Ondrej Certik <ond...@certik.cz>

i.e. around line 932:

if myFlag == True:
    expr = comms_final+noncomms_final
return Mul(coeff_self/coeff_old, Mul(*expr)._eval_subs(old,new))#*[e._eval_subs(old,new) for e in expr])

to the commented line, it works, the test passes, but a test related to the commutative substitution fails. (The bug was that comms_final was growing very large with the same Wild(). Or dummy symbol? It was named _p.)

I did not yet try to understand the code and find a real fix. I suspect "myFlag" is related. (We should really rename this one, btw.)

Please note that this bug was not triggered by the commit mentioned above, as this only occurred recently.

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To post to this group, send email to sympy-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
sympy-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/sympy-issues?hl=en.

Reply via email to