Skip to content

Conversation

@reidy-p
Copy link
Contributor

@reidy-p reidy-p commented Sep 13, 2018

@pep8speaks
Copy link

Hello @reidy-p! Thanks for submitting the PR.

- Bug in :class:`DatetimeIndex` subtraction that incorrectly failed to raise ``OverflowError`` (:issue:`22492`, :issue:`22508`)
- Bug in :class:`DatetimeIndex` incorrectly allowing indexing with ``Timedelta`` object (:issue:`20464`)
-
- Bug in :class:`DatetimeIndex` where addition of Week offset with weekday=6 had no effect (:issue:`22465`)
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this bug present in 0.23.x? I think it wasn't, in which case this entry can be removed as users never saw it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No I don't think this bug was in 0.23.x because the changes causing this bug are also part of 0.24.0 so I guess we can remove this whatsnew entry

Copy link
Member

Choose a reason for hiding this comment

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

Tests are passing. Don't forget to do this! Either @reidy-p or @TomAugspurger

@TomAugspurger
Copy link
Contributor

TomAugspurger commented Sep 13, 2018

Great. Since CI is almost done passing, we can just remove it when merging. Thanks for following up on this!

@gfyoung gfyoung added Bug Datetime Datetime data dtype Timedelta Timedelta data type labels Sep 14, 2018
@TomAugspurger TomAugspurger merged commit 59cfd8c into pandas-dev:master Sep 14, 2018
@TomAugspurger
Copy link
Contributor

Thanks!

@reidy-p reidy-p deleted the weekday6_addition branch September 15, 2018 14:25
aeltanawy pushed a commit to aeltanawy/pandas that referenced this pull request Sep 20, 2018
…ev#22695)

* BUG: Incorrect addition of Week(weekday=6) to DatetimeIndex
Sup3rGeo pushed a commit to Sup3rGeo/pandas that referenced this pull request Oct 1, 2018
…ev#22695)

* BUG: Incorrect addition of Week(weekday=6) to DatetimeIndex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Datetime Datetime data dtype Timedelta Timedelta data type

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change in behavior of DatetimeIndex + Offset

4 participants