Skip to content

Conversation

@jbrockmendel
Copy link
Member

  • closes #xxxx (Replace xxxx with the Github issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

@jreback jreback added the Non-Nano datetime64/timedelta64 with non-nanosecond resolution label Jun 5, 2022
@jreback jreback added this to the 1.5 milestone Jun 5, 2022
from pandas.core.arrays.timedeltas import TimedeltaArray

if self._ndarray.dtype != "M8[ns]":
raise NotImplementedError("Only supported for nanosecond resolution.")
Copy link
Contributor

Choose a reason for hiding this comment

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

any test hits this?

Copy link
Member Author

Choose a reason for hiding this comment

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

no, but this is deprecated anyway. mostly a note to myself not to bother implementing it for non-nano

@jreback jreback merged commit e9005c1 into pandas-dev:main Jun 10, 2022
@jbrockmendel jbrockmendel deleted the nano-dta branch June 11, 2022 01:21
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Non-Nano datetime64/timedelta64 with non-nanosecond resolution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants