-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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: Assorted DTA/TDA/PI bugfixes #24397
Conversation
Codecov Report
@@ Coverage Diff @@
## master #24397 +/- ##
===========================================
- Coverage 92.3% 42.99% -49.32%
===========================================
Files 162 162
Lines 51875 51886 +11
===========================================
- Hits 47883 22308 -25575
- Misses 3992 29578 +25586
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #24397 +/- ##
==========================================
- Coverage 92.3% 92.3% -0.01%
==========================================
Files 163 163
Lines 51946 51958 +12
==========================================
+ Hits 47949 47959 +10
- Misses 3997 3999 +2
Continue to review full report at Codecov.
|
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.
minor code comments. can you add whatsnews as needed.
is this orthogonal to #23078, should close that one? |
i think comments have been addressed |
thanks. pls followup on the ops name tests at some point. |
Fixes a problem with tz-aware DatetimeArray.round (ref #24038)
Fixes PI, PA, DTA failing to validate lengths when doing comparisons against length-1 instances (ref #23078)
Pins names to a handful of methods that currently don't have them (working on a separate branch that checks these systematically; these are the low-hanging fruit)
git diff upstream/master -u -- "*.py" | flake8 --diff