Both fraction and as_numer_denom return (2x+2, sin(x)+1) for expression 2(x+1)/(sin(x)+1), i.e. they rewrite numerator and denominator. I suggest simple fix for fraction: substitute last line return Mul(*numer), Mul(*denom) with return Mul(*numer, evaluate=False), Mul(*denom, evaluate=False) For as_numer_denom fix seems harder to implement.
Is it possible to change them? On Friday, March 27, 2015 at 1:40:39 AM UTC+2, Aaron Meurer wrote: > > It looks like fraction takes into account assumptions, but on the > other hand, it doesn't do any rewriting of the expression to combine > sums of fractions (e.g., fraction(1/x + 1/y) gives (1/x + 1/y, 1), > whereas (1/x + 1/y).as_numer_denom() gives (x + y, x*y)). > > Aaron Meurer > > On Thu, Mar 26, 2015 at 12:09 PM, Paul Royik <[email protected] > <javascript:>> wrote: > > I used as_numer_denom earlier and now found fraction. > > Is there any difference between them? > > > > -- > > You received this message because you are subscribed to the Google > Groups > > "sympy" group. > > To unsubscribe from this group and stop receiving emails from it, send > an > > email to [email protected] <javascript:>. > > To post to this group, send email to [email protected] > <javascript:>. > > Visit this group at http://groups.google.com/group/sympy. > > To view this discussion on the web visit > > > https://groups.google.com/d/msgid/sympy/64b73b09-213c-4735-a80b-f5a37a4c2766%40googlegroups.com. > > > > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "sympy" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sympy. To view this discussion on the web visit https://groups.google.com/d/msgid/sympy/546f988c-4db5-479d-b435-a6b70d3c5439%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
