Comment #7 on issue 2003 by asmeurer: Use factorint() with limit argument in sympy/core/numbers.py
http://code.google.com/p/sympy/issues/detail?id=2003

I haven't actually thoroughly read the factorint() code, but from what I read in the docsrting, the point of the limit argument is that it does try the more advanced methods, yes, but is limited by the limit argument, so speed shouldn't be an issue. Now, I don't know if the same number that was being used as the whole number limit (4294967296) should be used as the argument to limit in factorint(). For that, we should do some speed testing with numbers that are products of large primes to see how the timing goes.

By the way, we need to completely remove the trial factors routine in numbers.py, because it is duplicated by factorint(), and I think the factorint() version is more efficient anyway.

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

Reply via email to