BUG: GH15426 timezone lost in groupby-agg with cython functions #15433
Conversation
Codecov Report
@@ Coverage Diff @@ ## master #15433 +/- ## ========================================== + Coverage 90.36% 90.37% +<.01% ========================================== Files 136 136 Lines 49553 49554 +1 ========================================== + Hits 44780 44782 +2 + Misses 4773 4772 -1
Continue to review full report at Codecov.
|
| assert_frame_equal(result, expected) | ||
| | ||
| def test_agg_time_zone_round_trip(self): | ||
| ts = pd.Timestamp("2016-01-01 12:00:00", tz='US/Pacific') |
There was a problem hiding this comment.
add the issue number as a comment
| df = pd.DataFrame({'a': 1, 'b': [ts + timedelta(minutes=nn) | ||
| for nn in range(10)]}) | ||
| | ||
| result1 = df.groupby('a')['b'].agg(np.min).iloc[0] |
There was a problem hiding this comment.
compare with .min() as well
| # convert to datetime and change timezone | ||
| from pandas import to_datetime | ||
| result = to_datetime(result).tz_localize(dtype.tz) | ||
| result = to_datetime(result).tz_localize('utc') |
There was a problem hiding this comment.
can add addtl tests in pandas/tests/types/test_cast.py fit them in as appropriate. take out this fix temporarily to validate your test cases, then add back to make sure they work.
| add examples from #10668 as well. (and you can list list issue in the whatsnew too) |
| can you update |
stephenrauch force-pushed the tz-lost-in-groupby-agg branch from ae0ea81 to 64a84ca Compare | thanks! |
closes pandas-dev#15426 Author: Stephen Rauch <stephen.rauch+github@gmail.com> Closes pandas-dev#15433 from stephenrauch/tz-lost-in-groupby-agg and squashes the following commits: 64a84ca [Stephen Rauch] BUG: GH15426 timezone lost in groupby-agg with cython functions
3 participants
git diff upstream/master | flake8 --diff