-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
BUG: Assorted DatetimeIndex bugfixes #24157
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
092b3b9
Deprecate time_rule
jbrockmendel fbcc04b
cleanup
jbrockmendel ac52857
fix test
jbrockmendel 121c373
Fix indexing with ellipsis, GH#21282
jbrockmendel ef66bb9
isort tests.arithmetic, tests.dtypes
jbrockmendel 91738d3
raise on date_range with NaT for either start or end
jbrockmendel bb0d065
revert noncentral changes
jbrockmendel ff3a5c0
more useful error messages on validate_frequency failure
jbrockmendel e54159d
Fix DTI-->Categorical -->DTI roundtrip
jbrockmendel 44e0126
extend test to DTA
jbrockmendel 12e0f4e
comment
jbrockmendel 28bf2de
update GH reference
jbrockmendel 06d0a8e
make TDI catch nanos valuerror
jbrockmendel 81c7d0f
fix error message, hopefully
jbrockmendel 97f976f
Merge branch 'master' of https://github.com/pandas-dev/pandas into deets
jbrockmendel 9e55d97
Merge branch 'master' of https://github.com/pandas-dev/pandas into deets
jbrockmendel c7f280f
Merge branch 'master' of https://github.com/pandas-dev/pandas into deets
jbrockmendel af303c7
improve comments
jbrockmendel 8ece686
use take
jbrockmendel 30f01a0
whitespace
jbrockmendel 9617b85
Merge branch 'master' of https://github.com/pandas-dev/pandas into deets
jbrockmendel 3731098
Merge branch 'master' of https://github.com/pandas-dev/pandas into deets
jbrockmendel df05c88
do ellipsis check later
jbrockmendel 8f39b23
Avoid AssertionError
jbrockmendel e958ce6
remove resolved comment
jbrockmendel 97cb6b3
Merge branch 'master' of https://github.com/pandas-dev/pandas into deets
jbrockmendel 36d37e4
more whatsnew
jbrockmendel 3fc1c19
comment harder
jbrockmendel 88f7094
comment harder
jbrockmendel e178bb9
remove time_rule entirely
jbrockmendel 50f6b7e
whatsnew
jbrockmendel b927925
flake8 fixup
jbrockmendel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Merge branch 'master' of https://github.com/pandas-dev/pandas into deets
- Loading branch information
commit 9617b8511c0ccce6afe15d5a260b02c6649d999c
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
You are viewing a condensed version of this merge commit. You can view the full changes here.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
these don't jive with what's at the top of the PR. missing 2 notes?
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.
what about #11587 ?
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.
Does it merit a note? All that is changed is an error message. But yes, this does close 11587, I'll update this above.