-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
ENH add cython tutorial #3965
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
ENH add cython tutorial #3965
Conversation
I think you need to add this file to index.rst to get it 2 show up |
I'm looking forward to reading it. Do you think you could write up a guide
|
@y-p probably knows the most about caching builds since he's the one who added that functionality iirc. i'll try to help in the meantime...when u cache a build setup.py gets rewritten. i always do a |
#3156 describes the use of the build cache (I cleaned it up slightly last night). |
@@ -131,5 +131,6 @@ See the package overview for more detail about what's in the library. | |||
r_interface | |||
related | |||
comparison_with_r | |||
cython |
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.
i think you should put this after sparse (up a couple)
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.
Sure (as "Enhancing Performance").
mergable? |
@hayd btw pasting cell magics works in ipython master may want to note this and if you're crazy gung ho u can do a bisect and find out which change introduced this feature and point users the version that allows them to paste, but that's probably a bit overkill |
@jreback let me just make the change to Enhancing Performance (when I get home in an hour or so). It's not weird to have cython as the only section...? @cpcloud ah, that's strange, I thought I'd convinced myself pasting did work in my version, will check it out the ipython issues and see if I can locate change. Or maybe I'll say something vague, definitely worth mentioning. Actually... there is a lot of TODO's in there, was hoping for some feedback on those bits :) |
change name to Enhancing performance add in some sections
@jreback think it could be good to merge (with a once over) I've added sections and (recklessly) removed all the todos.... |
@hayd I think ok to merge.... I like to look at them on web for real...then fix things I find! |
let's do it then |
WIP #3923
Please checkout this draft for a pandas cython tutorial, any feedback or ideas appreciated.
Thanks to @cpcloud for patiently working out/explaining how to force ipython sphinx directive to play nicely with %%cython (it's incredibly sensitive!) and fixing the spacing bug. This seems to build now. :)