Conversation

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

Ran the hypothesis tests with max_examples=10**6

@simonjayhawkinssimonjayhawkins added Bug TimedeltaTimedelta data typeTimestamppd.Timestamp and associated methodslabels Feb 22, 2023
@@ -1210,12 +1210,14 @@ Datetimelike
- Bug in :func:`to_datetime` with both ``unit`` and ``origin`` specified returning incorrect results (:issue:`42624`)
- Bug in :meth:`GroupBy.quantile` with datetime or timedelta dtypes giving incorrect results for groups containing ``NaT`` (:issue:`51373`)
- Bug in :meth:`Groupby.quantile` incorrectly raising with :class:`PeriodDtype` or :class:`DatetimeTZDtype` (:issue:`51373`)
- Bug in :meth:`Timestamp.round` with values close to the implementation bounds returning incorrect results instead of raising ``OutOfBoundsDatetime`` (:issue:`51494`)
Copy link
Member

Choose a reason for hiding this comment

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

Does this impact the new non-nano feature? Otherwise might be good for 2.1 instead

Copy link
Member Author

Choose a reason for hiding this comment

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

Nothing specific to non-nano here, this will affect any reso

Copy link
Member

Choose a reason for hiding this comment

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

Got it. I think it may be safer to not backport for now and move to 2.1 unless you have a strong opinion for this to be in 2.0

Choose a reason for hiding this comment

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

Looks pretty good. Just a merge conflict

@jbrockmendel

rebased+greenish

@jbrockmendel

Note moved, greenish

@mroeschkemroeschke added this to the 2.1 milestone Feb 25, 2023
@mroeschkemroeschke merged commit efe896c into pandas-dev:main Feb 25, 2023
@mroeschke

Thanks @jbrockmendel

@jbrockmendeljbrockmendel deleted the tst-round-xfails branch February 25, 2023 02:24
Sign up for free to join this conversation on . Already have an account? Sign in to comment
Bug TimedeltaTimedelta data typeTimestamppd.Timestamp and associated methods
None yet

Successfully merging this pull request may close these issues.