-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC: GH10414 Missing example in NA values in GroupBy #10450
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
DOC: GH10414 Missing example in NA values in GroupBy #10450
Conversation
@harshnisar see the contributing docs here pls pull current master and rebase. I know its only a trivial change, but this is in conflict as master changed a tiny bit in this section a couple of days ago. |
485e63e
to
6e10194
Compare
Done :) |
See the groupby section :ref:`here <groupby.missing>` for more information. | ||
======= | ||
.. ipython:: python |
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.
you need to remove the conflict lines
further I would move your change above the see further reference
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.
@jreback
Embarrassing. Sorry.
6e10194
to
925d6f3
Compare
|
||
See the groupby section :ref:`here <groupby.missing>` for more information. | ||
======= |
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 this snuck in during a rebase/merge?
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.
@shoyer In my defence, it really did fit well. Changed 👍 Embarrassed.
Looks good now! @harshnisar Last question: can you squash the three commits into one? (see end of this section: http://pandas.pydata.org/pandas-docs/stable/contributing.html#committing-your-code) |
80c241d
to
46def53
Compare
46def53
to
204ce41
Compare
@jorisvandenbossche I know by now, this pull request is more of an effort sink for you than a contribution by me. Won't happen from next pulls. |
@harshnisar no, no problem! And thanks! |
DOC: GH10414 Missing example in NA values in GroupBy
closes #10414