Re: Issue 3322 in sympy: Another CSE bug

2012-07-11 Thread sympy
Updates: Status: Fixed Comment #10 on issue 3322 by smi...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 Thanks for reporting this, Andreas. This works now. -- You received this message because you are subscribed to the Google Groups "sympy-issues" gro

Re: Issue 3322 in sympy: Another CSE bug

2012-07-10 Thread sympy
Updates: Labels: NeedsReview smichr Comment #9 on issue 3322 by smi...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 There is a fix at 1409 -- the failure of -(-1)**(1/4)*I to combine into -(-1)**(3/4) is reason enough not to tweak as_base_exp. Tests a

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #8 on issue 3322 by andreas.kloeck...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 should've said in comment 6: does *not* try to compute (-1)**(3/2). -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. T

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #7 on issue 3322 by andreas.kloeck...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 (Which, btw, would also already have been a fix for issue 3262.) -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To p

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #6 on issue 3322 by andreas.kloeck...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 My issue along with 3022 is fixed by changing this in ImaginaryUnit: def as_base_exp(self): -return S.NegativeOne, S.Half +return S.ImaginaryUnit, S

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #5 on issue 3322 by asmeu...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 No, the test failure I'm taling about is related to cse. See the test_issue_3022 in test_expand.py. -- You received this message because you are subscribed to the Google Group

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #4 on issue 3322 by andreas.kloeck...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 Not sure how related issue 3022 is, as it is about expand(). There is however an eerie similarity with http://code.google.com/p/sympy/issues/detail?id=3262#c2 Mul._eval

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #3 on issue 3322 by asmeu...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 The problems with the test failure in master related to issue 3022 are also related to cse() and I, so perhaps these should both be investigated together. -- You received th

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #2 on issue 3322 by andreas.kloeck...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 Even simpler: --- import sympy as sp from sympy import I z = - 2*I x0 = sp.Symbol("x0") two_i = 2*I print z._subs(t

Re: Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Comment #1 on issue 3322 by andreas.kloeck...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 Reduced to bug in subs: --- import sympy as sp from sympy import exp, I z = exp(2*I) - 2*I print z.subs({2*I: sp.Sy

Issue 3322 in sympy: Another CSE bug

2012-07-09 Thread sympy
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3322 by andreas.kloeck...@gmail.com: Another CSE bug http://code.google.com/p/sympy/issues/detail?id=3322 Hi there, the following program: - import sympy as sp from sympy impor