Re: Issue 2061 in sympy: sqrt(-1.0*x) gives infinite recursion

2012-01-24 Thread sympy
Updates: Status: Fixed Comment #6 on issue 2061 by smi...@gmail.com: sqrt(-1.0*x) gives infinite recursion http://code.google.com/p/sympy/issues/detail?id=2061 (No comment was entered for this change.) -- You received this message because you are subscribed to the Google Groups

Re: Issue 2061 in sympy: sqrt(-1.0*x) gives infinite recursion

2012-01-22 Thread sympy
Updates: Labels: NeedsReview smichr Comment #5 on issue 2061 by smi...@gmail.com: sqrt(-1.0*x) gives infinite recursion http://code.google.com/p/sympy/issues/detail?id=2061 see https://github.com/sympy/sympy/pull/1013 -- You received this message because you are subscribed to the

Re: Issue 2061 in sympy: sqrt(-1.0*x) gives infinite recursion

2012-01-13 Thread sympy
Updates: Status: Accepted Comment #4 on issue 2061 by smi...@gmail.com: sqrt(-1.0*x) gives infinite recursion http://code.google.com/p/sympy/issues/detail?id=2061 The result above is not correct: if x == -1, sqrt(-1.*x) should give 1.0. Instead, expanding the result as

Re: Issue 2061 in sympy: sqrt(-1.0*x) gives infinite recursion

2011-05-14 Thread sympy
Updates: Status: Fixed Comment #3 on issue 2061 by smi...@gmail.com: sqrt(-1.0*x) gives infinite recursion http://code.google.com/p/sympy/issues/detail?id=2061 sqrt(-1.*x) 1.0*I*x**(1/2) -- You received this message because you are subscribed to the Google Groups

Issue 2061 in sympy: sqrt(-1.0*x) gives infinite recursion

2010-09-21 Thread sympy
Status: Accepted Owner: smichr Labels: Type-Defect Priority-Medium New issue 2061 by smichr: sqrt(-1.0*x) gives infinite recursion http://code.google.com/p/sympy/issues/detail?id=2061 This happens because a test for coeff is S.One fails since coeff is Real(1). as_coeff_terms/factors could be

Re: Issue 2061 in sympy: sqrt(-1.0*x) gives infinite recursion

2010-09-21 Thread sympy
Updates: Labels: -Priority-Medium Priority-High Comment #1 on issue 2061 by asmeurer: sqrt(-1.0*x) gives infinite recursion http://code.google.com/p/sympy/issues/detail?id=2061 Maybe you could introduce as_rational_coeff_terms/factors(). But are you sure that that is the correct fix

Re: Issue 2061 in sympy: sqrt(-1.0*x) gives infinite recursion

2010-09-21 Thread sympy
Comment #2 on issue 2061 by smichr: sqrt(-1.0*x) gives infinite recursion http://code.google.com/p/sympy/issues/detail?id=2061 Maybe overkill. If all tests pass with just treating leading Rationals rather than Numbers then perhaps that's enough of a change. The only nuance is that whereas