Comment #53 on issue 2607 by [email protected]: as_numer_denom() is too slow
http://code.google.com/p/sympy/issues/detail?id=2607
This is another level of changes that need to be made in order to
support hints. If expansion is desired then the better routine should
be used rather than just expanding at the end of the routine. I'm not
sure I'm up to this. I've got a lot of work that is sitting on the
docks and not moving already.
Also, only 1 test failed with the binary method, and it did so pretty
trivially. So perhaps it's good enough to just leave the binary method
as the default method.
Here are the results of the different routines on a small expression
(from three different runs where the default method calls the named
method given as the title of each section below:
binary
(a*b*c*z*(x + 1) + d*(a*z*(b + c) + b*c*(a*y + z)), a*b*c*d*z)
(1008*a + 840*b + 720*c + 2520*x + 1680*y + 1260*z, 5040)
(a + b + c + x + y + z, q)
expanded
(a*b*c*d**2*y + a*b*c*d*x*z + a*b*c*d*z + a*b*d**2*z + a*c*d**2*z +
b*c*d**2*z,
a*b*c*d**2*z)
(1008*a + 840*b + 720*c + 2520*x + 1680*y + 1260*z, 5040)
(a*q**5 + b*q**5 + c*q**5 + q**5*x + q**5*y + q**5*z, q**6)
collect
(a*b*c*d*y + a*b*c*z*(x + 1) + a*b*d*z + a*c*d*z + b*c*d*z, a*b*c*d*z)
(84*a + 70*b + 60*c + 210*x + 140*y + 105*z, 420)
(a + b + c + x + y + z, q)
--
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.
