Resampling with NaT in TimedeltaIndex raises MemoryError #22253
Conversation
doc/source/whatsnew/v0.24.0.txt
Outdated
| - Multiple bugs in :func:`pandas.core.Rolling.min` with ``closed='left'` and a | ||
| datetime-like index leading to incorrect results and also segfault. (:issue:`21704`) | ||
| - | ||
| - Bug in :meth:`DataFrame.resample` when resampling NaT in `TimeDeltaIndex` (:issue:`13223`). |
There was a problem hiding this comment.
use double-backticks on NaT and on TDI
There was a problem hiding this comment.
I would take this in 0.23.5
| assert_frame_equal(result, expected) | ||
|
|
||
| def test_resampling_with_nat(self): | ||
| # GH 13223 |
There was a problem hiding this comment.
can you see if you move this to Base, so it runs on DTI, Period, and DTI (use self.create_series() then add a NaT).
alternatively you can just write separate tests for DTI and Period (we may have for DTI already), pls indicate if so.
There was a problem hiding this comment.
k thanks, if you are interested in consolidating these into Base (new PR) would be great (if its easy)
Codecov Report
@@ Coverage Diff @@
## master #22253 +/- ##
==========================================
- Coverage 92.07% 92.07% -0.01%
==========================================
Files 169 169
Lines 50684 50682 -2
==========================================
- Hits 46668 46666 -2
Misses 4016 4016
Continue to review full report at Codecov.
|
2 participants
git diff upstream/master -u -- "*.py" | flake8 --diff