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

TST: Groupby.groups of datetimeindex (#11442) #14952

Merged
merged 1 commit into from
Dec 23, 2016

Conversation

mroeschke
Copy link
Member

Doesn't look a PR in 0.19.2 and 0.20.0 addressed this, but it works on master 0.19.1

@codecov-io
Copy link

codecov-io commented Dec 22, 2016

Current coverage is 84.65% (diff: 100%)

Merging #14952 into master will decrease coverage by <.01%

@@             master     #14952   diff @@
==========================================
  Files           144        144          
  Lines         51030      51030          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          43199      43198     -1   
- Misses         7831       7832     +1   
  Partials          0          0          

Powered by Codecov. Last update 74de478...6c16211

@jreback jreback added Groupby Testing pandas testing functions or related to the test suite labels Dec 22, 2016
@jreback jreback added this to the 0.20.0 milestone Dec 22, 2016
@jreback jreback added the Bug label Dec 22, 2016
result = df.groupby(level='date').groups
dates = ['2015-01-05', '2015-01-04', '2015-01-03',
'2015-01-02', '2015-01-01']
expected = {pd.Timestamp(date): pd.DatetimeIndex([date], name='date')
Copy link
Contributor

Choose a reason for hiding this comment

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

add the .get_group call as well here. ping on green.

@jreback jreback merged commit 9accef8 into pandas-dev:master Dec 23, 2016
@jreback
Copy link
Contributor

jreback commented Dec 23, 2016

thanks!

ShaharBental pushed a commit to ShaharBental/pandas that referenced this pull request Dec 26, 2016
@mroeschke mroeschke deleted the test_11442 branch December 20, 2017 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Groupby Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Groupby.groups doesn't work by a groups convert from DateTimeIndex
3 participants