-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DEPS: bump xlrd min version to 1.0.0 #23774
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
Conversation
removes all of those annoying warnings on 2.7 build :> |
Codecov Report
@@ Coverage Diff @@
## master #23774 +/- ##
=========================================
Coverage ? 92.28%
=========================================
Files ? 161
Lines ? 51451
Branches ? 0
=========================================
Hits ? 47483
Misses ? 3968
Partials ? 0
Continue to review full report at Codecov.
|
Hello @jreback! Thanks for updating the PR.
|
@@ -652,7 +653,8 @@ def get_value(self, series, key): | |||
except TypeError: | |||
pass | |||
|
|||
key = Period(key, self.freq).ordinal | |||
period = Period(key, self.freq) | |||
key = period.value if isna(period) else period.ordinal |
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.
cc @jbrockmendel I had to make this change here. Not sure how this actually worked before.
remove numpy warnings on np.diff in boolean arrays
@TomAugspurger if any comments |
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.
Not sure about the period change (the change itself looks fine / necessary. Not sure why it wasn't needed / working before).
…fixed * upstream/master: (46 commits) DEPS: bump xlrd min version to 1.0.0 (pandas-dev#23774) BUG: Don't warn if default conflicts with dialect (pandas-dev#23775) BUG: Fixing memory leaks in read_csv (pandas-dev#23072) TST: Extend datetime64 arith tests to array classes, fix several broken cases (pandas-dev#23771) STYLE: Specify bare exceptions in pandas/tests (pandas-dev#23370) ENH: between_time, at_time accept axis parameter (pandas-dev#21799) PERF: Use is_utc check to improve performance of dateutil UTC in DatetimeIndex methods (pandas-dev#23772) CLN: io/formats/html.py: refactor (pandas-dev#22726) API: Make Categorical.searchsorted returns a scalar when supplied a scalar (pandas-dev#23466) TST: Add test case for GH14080 for overflow exception (pandas-dev#23762) BUG: Don't extract header names if none specified (pandas-dev#23703) BUG: Index.str.partition not nan-safe (pandas-dev#23558) (pandas-dev#23618) DEPR: tz_convert in the Timestamp constructor (pandas-dev#23621) PERF: Datetime/Timestamp.normalize for timezone naive datetimes (pandas-dev#23634) TST: Use new arithmetic fixtures, parametrize many more tests (pandas-dev#23757) REF/TST: Add more pytest idiom to parsers tests (pandas-dev#23761) DOC: Add ignore-deprecate argument to validate_docstrings.py (pandas-dev#23650) ENH: update pandas-gbq to 0.8.0, adds credentials arg (pandas-dev#23662) DOC: Improve error message to show correct order (pandas-dev#23652) ENH: Improve error message for empty object array (pandas-dev#23718) ...
remove numpy warnings on np.diff in boolean arrays