Updates:
        Cc: Ronan.Lamy smichr Vinzent.Steinberg

Comment #10 on issue 1985 by asmeurer: as_real_imag() gives wrong answer when expanding quotient
http://code.google.com/p/sympy/issues/detail?id=1985

So I was so convinced that my solution (refactoring expand(complex=True) and as_real_imag()) was so much better that I went ahead and created a patch for it. Now as_real_imag should be much more infallible, because it does not rely on pattern matching to take the real and imaginary parts, but rather uses the logic that was already there in expand(complex=True).

Note that some of the things in this patch will need to be refactored to use Chris's expand hint system (like expand_multinomial). Also, patching expand functions feels like putting a bandaid on a 4 in. flesh wound. This whole system needs to be refactored to use the new polys and to not be so redundant and sensitive to infinite recursion.

See http://github.com/asmeurer/sympy/tree/1985

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

Reply via email to