#9451: [with patch, needs work] sieve of atkin
-----------------------------+----------------------------------------------
Reporter: rohana | Owner: was
Type: enhancement | Status: new
Priority: major | Milestone: sage-5.0
Component: number theory | Keywords: prime, sieve, range
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------------+----------------------------------------------
Comment(by mhansen):
A couple quick things without really looking at the content of the patch:
1) You should probably import prime_range into fast_arith for backward
compatibility.
2) You don't need backslashes to continue lines when they're in brackets.
3) You should make the default algorithm {{{None}}} and choose it inside
of the function. That way it can choose a different algorithm if the
input is outside of the range of atkins.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9451#comment:2>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" 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/sage-trac?hl=en.
