-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
BUG: Fix date_range overflow #23345
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BUG: Fix date_range overflow #23345
Conversation
Hello @jbrockmendel! Thanks for submitting the PR.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm comment on the issue number
doc/source/whatsnew/v0.24.0.txt
Outdated
@@ -1003,6 +1003,7 @@ Datetimelike | |||
- Bug in :func:`to_datetime` with an :class:`Index` argument that would drop the ``name`` from the result (:issue:`21697`) | |||
- Bug in :class:`PeriodIndex` where adding or subtracting a :class:`timedelta` or :class:`Tick` object produced incorrect results (:issue:`22988`) | |||
- Bug in :func:`date_range` when decrementing a start date to a past end date by a negative frequency (:issue:`23270`) | |||
- Bug in :func:`date_range` with frequency of ``Day`` or higher where dates sufficiently far in the future could wrap around to the past instead of raising ``OutOfBoundsDatetime`` (:issue:`19740`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a closed PR is this the right number?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe #14187 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, fixed.
Codecov Report
@@ Coverage Diff @@
## master #23345 +/- ##
=========================================
Coverage ? 92.22%
=========================================
Files ? 169
Lines ? 51267
Branches ? 0
=========================================
Hits ? 47283
Misses ? 3984
Partials ? 0
Continue to review full report at Codecov.
|
can you rebase |
thanks! |
git diff upstream/master -u -- "*.py" | flake8 --diff