Status: Accepted
Owner: smichr
CC: ondrej.certik,  asmeurer,  Vinzent.Steinberg
Labels: Type-Defect Priority-Medium NeedsReview

New issue 1918 by smichr: 1918: sqrt speed up
http://code.google.com/p/sympy/issues/detail?id=1918

sqrt had a large time penalty associated with it because of
the extra symping that was occuring and because of it using
** instead of Pow.

timeit('sqrt(1)', 'from sympy import S, sqrt')
20.878364149633541
timeit('1**S.Half', 'from sympy import S, sqrt')
14.745359815283763
timeit('Pow(1, S.Half)', 'from sympy import S, sqrt, Pow')
3.2267961430852097

After this patch, it can be used with less time-worries as
sugar for Pow(x, 1/2):

timeit('sqrt(1)', 'from sympy import sqrt')
3.983598194742628

This is commit 1918 in smichr's 1766 branch at github.

--
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.

Reply via email to