Skip to content

Typed cpdef() #3

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Typed cpdef() #3

wants to merge 24 commits into from

Conversation

itziakos
Copy link
Contributor

@itziakos itziakos commented May 9, 2015

This PR updates the notes to include benchmarks for using a typed cpdef function for the Fibonacci function (recursive algorithm).

in more details:

  • All typed functions now return long to be consistent with the C implementation of the fibonacci function.
  • A new bechmark script is provided
  • Documentation is updated with the benchmarking of the typed cpdef (it is as fast or even faster than the hand written C wrapper.
  • A simple script is provided to create the comparison graph.
  • A testcase is added for the fibonacci functions.
  • A make.bat is provided for compiling the documentation under windows

itziakos added 24 commits May 1, 2015 09:00
@itziakos
add a typed cpdef function
f42884b
@itziakos
add a testcase for the fibonacci functions
d137a14
@itziakos
fix: support both int and long
ac67f70
@itziakos
use long for consistancy
62d82fe
@itziakos
add cpdef_int to timeit
419c13d
@itziakos
add the handwraped extension into the timeit commands
25988a2
@itziakos
run the tests on travis-ci
250d060
@itziakos
fix test command
45ae207
@itziakos
better support for python2 and python 3 checks
f9cebb9
@itziakos
use a bench script
81459d6
@itziakos
minor cleanup
5f6b8f5
@itziakos
add batch file for documentation build
009c1a6
@itziakos
wip commit
dcaf046
@itziakos
use the console_timeit
bbff42e
@itziakos
add code sample for the typed cpdef
8f5bdfd
@itziakos
add small script to create the fib comparison plot
67195a1
@itziakos
update info on running the bench marks
6e347ab
@itziakos
use a literalinclude directive
8f3c364
@itziakos
update table based on recent runs in travis-ci
4ef787b
@itziakos
update discussion and layout
1537656
@itziakos
update comparison graph
89a57e0
@itziakos
Update makeplot.py
64a4931
@itziakos
Merge pull request #2 from itziakos/typed-cpdef
498e4b4 
Typed cpdef
@itziakos
Update fibo_speed.rst
b1728b5
itziakos
itziakos reviewed May 9, 2015
View reviewed changes
doc/source/fibo_speed.rst
@@ -207,12 +140,16 @@ Or, graphically:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will need to also update the table and graphs about the cached based function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@itziakos