Comment #7 on issue 1736 by ronan.l...@gmail.com: Refactor pattern matching and substitution http://code.google.com/p/sympy/issues/detail?id=1736
"Why do you use Rational(1) instead of S.one?" For no good reason. It was just convenient to replace Rational(sign) with Rational(1). This part of the code probably needs further refactoring anyway. "What is the motivation of atomic_subs()?" The idea is to perform purely structural substitution - i.e. considering only the expression tree and not at all the mathematical meaning. The need for this has been mentioned several times already, most recently in issue 1520 and also in issue 666. -- 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 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.