Skip to content

Commit 20c93d2

Browse files
fix raise of TypeError when subtracting timedelta array
closes #21980
1 parent e2e1a10 commit 20c93d2

File tree

3 files changed

+62
-4
lines changed

3 files changed

+62
-4
lines changed

doc/source/whatsnew/v0.24.0.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -594,7 +594,7 @@ Timedelta
594594
- Bug in multiplying a :class:`Series` with numeric dtype against a ``timedelta`` object (:issue:`22390`)
595595
- Bug in :class:`Series` with numeric dtype when adding or subtracting an an array or ``Series`` with ``timedelta64`` dtype (:issue:`22390`)
596596
- Bug in :class:`Index` with numeric dtype when multiplying or dividing an array with dtype ``timedelta64`` (:issue:`22390`)
597-
-
597+
- Fixed bug where subtracting :class:`Timedelta` from an object-dtyped array would raise ``TypeError`` (:issue:`21980`)
598598
-
599599
-
600600

pandas/_libs/tslibs/timedeltas.pyx

+5-3
Original file line numberDiff line numberDiff line change
@@ -541,10 +541,12 @@ def _binary_op_method_timedeltalike(op, name):
541541

542542
elif hasattr(other, 'dtype'):
543543
# nd-array like
544-
if other.dtype.kind not in ['m', 'M']:
545-
# raise rathering than letting numpy return wrong answer
544+
if other.dtype.kind in ['m', 'M']:
545+
return op(self.to_timedelta64(), other)
546+
elif other.dtype.kind == 'O':
547+
return np.array([op(self, x) for x in other])
548+
else:
546549
return NotImplemented
547-
return op(self.to_timedelta64(), other)
548550

549551
elif not _validate_ops_compat(other):
550552
return NotImplemented

pandas/tests/scalar/timedelta/test_arithmetic.py

+56
Original file line numberDiff line numberDiff line change
@@ -200,6 +200,48 @@ def test_td_rsub_numeric_raises(self):
200200
with pytest.raises(TypeError):
201201
2.0 - td
202202

203+
def test_td_sub_timedeltalike_object_dtype_array(self):
204+
# GH 21980
205+
arr = np.array([Timestamp('20130101 9:01'),
206+
Timestamp('20121230 9:02')])
207+
exp = np.array([Timestamp('20121231 9:01'),
208+
Timestamp('20121229 9:02')])
209+
res = arr - pd.Timedelta('1D')
210+
tm.assert_numpy_array_equal(res, exp)
211+
212+
def test_td_sub_mixed_most_timedeltalike_object_dtype_array(self):
213+
# GH 21980
214+
now = pd.Timestamp.now()
215+
arr = np.array([now,
216+
pd.Timedelta('1D'),
217+
np.timedelta64(2, 'h')])
218+
exp = np.array([now - pd.Timedelta('1D'),
219+
pd.Timedelta('0D'),
220+
np.timedelta64(2, 'h') - pd.Timedelta('1D')])
221+
res = arr - pd.Timedelta('1D')
222+
tm.assert_numpy_array_equal(res, exp)
223+
224+
@pytest.mark.parametrize('op', [operator.add, ops.radd])
225+
def test_td_add_timedeltalike_object_dtype_array(self, op):
226+
# GH 21980
227+
arr = np.array([Timestamp('20130101 9:01'),
228+
Timestamp('20121230 9:02')])
229+
exp = np.array([Timestamp('20130102 9:01'),
230+
Timestamp('20121231 9:02')])
231+
res = op(arr, pd.Timedelta('1D'))
232+
tm.assert_numpy_array_equal(res, exp)
233+
234+
@pytest.mark.parametrize('op', [operator.add, ops.radd])
235+
def test_td_add_mixed_timedeltalike_object_dtype_array(self, op):
236+
# GH 21980
237+
now = pd.Timestamp.now()
238+
arr = np.array([now,
239+
pd.Timedelta('1D')])
240+
exp = np.array([now + pd.Timedelta('1D'),
241+
pd.Timedelta('2D')])
242+
res = op(arr, pd.Timedelta('1D'))
243+
tm.assert_numpy_array_equal(res, exp)
244+
203245

204246
class TestTimedeltaMultiplicationDivision(object):
205247
"""
@@ -616,3 +658,17 @@ def test_rdivmod_invalid(self):
616658

617659
with pytest.raises(TypeError):
618660
divmod(np.array([22, 24]), td)
661+
662+
@pytest.mark.parametrize('op', [
663+
operator.mul,
664+
ops.rmul,
665+
operator.truediv,
666+
ops.rdiv,
667+
ops.rsub])
668+
@pytest.mark.parametrize('arr', [
669+
np.array([Timestamp('20130101 9:01'), Timestamp('20121230 9:02')]),
670+
np.array([pd.Timestamp.now(), pd.Timedelta('1D')])
671+
])
672+
def test_td_op_timedelta_timedeltalike_array(self, op, arr):
673+
with pytest.raises(TypeError):
674+
op(arr, pd.Timedelta('1D'))

0 commit comments

Comments
 (0)