Closed sjvrijn closed 8 years ago
Thanks for the comments, I'll fix them and create a new pull request
I think that if you update though branch the pull request should update automatically, no need to open a new one.
On 29 February 2016 10:17:32 pm AEDT, Sander notifications@github.com wrote:
Thanks for the comments, I'll fix them and create a new pull request
Reply to this email directly or view it on GitHub: https://github.com/naught101/sobol_seq/pull/4#issuecomment-190161193
Sent from my Android device with K-9 Mail. Please excuse my brevity.
Fixes are up and thanks for the tip, didn't realize that
Great, thank you :)
The calculations in
is_prime(n)
only have to run up tosqrt(n)
, and can skip any number of the form 2n and 3n, leaving those of the form 6n+1 and 6n+5. Cleanup:while (1)
withwhile True