Comment #2 on issue 2307 by asmeurer: Duplicate methods: as_coeff_mul and
as_coeff_Mul
http://code.google.com/p/sympy/issues/detail?id=2307
as_coeff_mul() is used all over the place. as_coeff_Mul() is only used in
a few places.
The docstring of as_coeff_Mul only says that it is there to efficiently get
the coefficient. But as_coeff_mul is faster:
In [71]: a = 3*Mul(*[Symbol('x%d' % i) for i in range(100)])
In [72]: %timeit a.as_coeff_mul()
100000 loops, best of 3: 10.4 us per loop
In [73]: %timeit a.as_coeff_Mul()
100000 loops, best of 3: 26.7 us per loop
So I vote to remove as_coeff_Mul(). A patch is coming soon, unless there
are any objections.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post to this group, send email to sympy-issues@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.
