Skip to content
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: Fix groupby nth #13810

Merged
merged 1 commit into from
Jul 29, 2016
Merged

DOC: Fix groupby nth #13810

merged 1 commit into from
Jul 29, 2016

Conversation

sinhrks
Copy link
Member

@sinhrks sinhrks commented Jul 27, 2016

  • tests added / passed
  • passes git diff upstream/master | flake8 --diff

Current API doc looks incorrect. Fixed it to meet #11039.

cc @behzadnouri

@codecov-io
Copy link

codecov-io commented Jul 27, 2016

Current coverage is 85.23% (diff: 100%)

Merging #13810 into master will increase coverage by <.01%

@@             master     #13810   diff @@
==========================================
  Files           140        140          
  Lines         50420      50420          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          42974      42976     +2   
+ Misses         7446       7444     -2   
  Partials          0          0          

Powered by Codecov. Last update 31f8e4d...948255b

@jorisvandenbossche
Copy link
Member

Can you show an example with as_index=False as well?

@sinhrks
Copy link
Member Author

sinhrks commented Jul 27, 2016

@jorisvandenbossche good idea. added.

@jreback jreback merged commit 2c55f28 into pandas-dev:master Jul 29, 2016
@jreback
Copy link
Contributor

jreback commented Jul 29, 2016

thanks!

@sinhrks sinhrks deleted the groupby_nth_doc branch July 29, 2016 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants