-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC: Flesh out the R comparison section of docs (GH3980) #5761
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
Thanks @chappers , this is useful stuff. But the PR violates a bunch of git-related guidelines
Both of these do nasy things to the git history and so the devs follow a bunch of rules on how to use git to avoid that. Please fix it yourself or let us know if you need git help. It's fine if you do, we all did at some point. |
@chappers - easiest is just to copy paste your changes to a separate file, checkout current master, and then paste them back on. Since it's docs and a relatively new section of the docs, you should be fine. |
Yeah...I don't have any idea what I'm doing. So I'm suppose to update a branch to upstream/master and then run |
No, that would have prevented this, now you need to fix the existing branch. upddate your master branch to point at current master on github, then
|
Thanks a lot. |
nicely done. |
Thanks for this work! |
@@ -5,6 +5,8 @@ | |||
:suppress: | |||
|
|||
from pandas import * | |||
import numpy.random as random | |||
from numpy import * |
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.
Can you import numpy as np? (and maybe also pandas as pd)
since this is only docs, as soon as @jorisvandenbossche , @y-p ok...then can merge it in |
I haven't run sphinx, but assuming it's ok... merge away. |
I ran Sphinx with this, and its OK. I cannot judge everything in the PR (the R code), but it seems OK. |
ok will merge and can check how |
DOC: Flesh out the R comparison section of docs (GH3980)
docs look good http://pandas.pydata.org/pandas-docs/dev/comparison_with_r.html feel free to add more :) |
Some additions to #3980.
I've done some more comparisons to the plyr functions located here. More specifically aaply, alply don't really have much to do with pandas (its just list comprehension). and dlply is probably more suited to be in statsmodel example rather than pandas.