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.

Sign up for GitHub

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

fix for BUG: grouping with tz-aware: Values falls after last bin #24973

Merged
merged 16 commits into from
Jan 29, 2019
Merged

fix for BUG: grouping with tz-aware: Values falls after last bin #24973

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
1940cfd
fix #24972
ahcub Jan 28, 2019
dd42200
improve comment
ahcub Jan 28, 2019
f8cb89a
add entry to what's new
ahcub Jan 28, 2019
eccca8e
add test for issue 24972
ahcub Jan 28, 2019
0454dc3
remove import
ahcub Jan 28, 2019
b009f17
change fix for values fall after last bin
ahcub Jan 28, 2019
66861ab
fix imports
ahcub Jan 28, 2019
dda1c5c
fix imports
ahcub Jan 28, 2019
3edb267
change the description
ahcub Jan 28, 2019
c0bc3a5
move test from groupby to timegrouper
ahcub Jan 28, 2019
e631821
fix typo
ahcub Jan 28, 2019
5c07d25
move tests
ahcub Jan 28, 2019
df7d650
chagne the bug fix description
ahcub Jan 29, 2019
bcc0599
Merge branch 'master' into master
ahcub Jan 29, 2019
9c7d0e1
Merge branch 'master' into master
ahcub Jan 29, 2019
47eb40b
remove the record that is not on master anymore
ahcub Jan 29, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
change the description
  • Loading branch information
@ahcub
ahcub committed Jan 28, 2019
commit 3edb267885e715aaf0577e1499bba2f66fb15a9d
2 changes: 1 addition & 1 deletion doc/source/whatsnew/v0.24.1.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ Bug Fixes
**Reshaping**

- Bug in :func:`merge` when merging by index name would sometimes result in an incorrectly numbered index (:issue:`24212`)
Bug in ``DataFrame.groupby(pd.Grouper(freq='1d'))`` when there is a time change (DST) and grouping frequecy is 1d (:issue:`24972`)
Bug in :meth:`DataFrame.groupby`(:class:`pd.Grouper`(freq='1d')) when there is a time change (DST) and grouping frequecy is 1d (:issue:`24972`)
Copy link
Member

Choose a reason for hiding this comment

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

frequecy --> frequency

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed, sorry

**Other**

-
Expand Down