-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
Shootout pidigits code, new shootout directory in examples and moved existing shootout code to it #700
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…e examples; it is a straighforward rewrite of the Python version available on the shootout. Also had to modify the bigint.jl module and corresponding gmp_wrapper.c in order to make 'divmod' and 'lshift' available. Without the printouts (commented out in the code), the speed is on par with Python -- use the timeit macro now available in timing.jl (also in shootout directory) and adapted from perf.jl. Also moved fasta.jl to that new shootout directory.
…Also: added some documentation for pidigits ; added an option to print the intermediate digits as requested by the shootout, leaving it as a non-default option because printing is so slow in Julia compared to Python...
@@ -0,0 +1,78 @@ | |||
#Assume running from julia base dir |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes no such assumption; timing.jl
is in .
and bigint.jl
will be on the search path regardless. You can drop this comment.
Someone also needs to find out if the Thanks @yledu, I think this is close pending the |
And note that since this is now a pull request on a branch (instead of a single commit), you can push new commits with any additional changes onto this branch on Github and they'll automagically show up here. |
Doesn't run right now but that's due to an unrelated issue with |
Shootout pidigits code, new shootout directory in examples and moved existing shootout code to it
Oops. I get the following error on
|
I hate our Makefiles. |
It doesn't show up on Linux because the .so DLL extension is correct there. On OS X it breaks. Easy fix. |
Oh, I saw the line had changed to get rid of the hardcoded path, but didn't realize it hadn't quite made it all the way back to where it started. D'oh! |
#693