Comment #11 on issue 1598 by asmeurer: New polynomials manipulation module http://code.google.com/p/sympy/issues/detail?id=1598
> This code was a part of Poly.cancel, which was copied from radsimp(). > However, > radsimp() tries to do more and is problematic (there are issues opened > for this OK. I didn't know that was from Poly.cancel(). Looking at the radsimp source, it looks like the only difference between your code and radsimp is that radsimp tries to handle the case a + b*sqrt(c) where b, c == 0 and it tries to collect terms. Maybe you could add a keyword argument to radsimp that turns this stuff off (like radsimp(expr, simple=False)), and then use that argument in the call in simplify(). That way, you are not duplicating code. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
