-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
tslib flake8 cleanup, missing declarations #17907
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
tslib flake8 cleanup, missing declarations #17907
Conversation
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.
looks fine. ping on green.
can you show updated benchmarks (for things that changed)
pandas/_libs/tslib.pyx
Outdated
@@ -1669,6 +1666,7 @@ cpdef convert_str_to_tsobject(object ts, object tz, object unit, | |||
cdef: | |||
_TSObject obj | |||
int out_local = 0, out_tzoffset = 0 | |||
datetime dtime |
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.
call as dt (like below)
Should just be Old:
New:
|
since it looks like you are doing flake8 fixups, can you edit the lint.sh script to pick these things up? |
Just added this to the TODO list. |
Codecov Report
@@ Coverage Diff @@
## master #17907 +/- ##
==========================================
- Coverage 91.23% 91.22% -0.02%
==========================================
Files 163 163
Lines 50105 50112 +7
==========================================
- Hits 45715 45713 -2
- Misses 4390 4399 +9
Continue to review full report at Codecov.
|
thanks @jbrockmendel |
replace
isinstance
checks with the faster C versionschange cpdef-->cdef for internal field-getter methods
remove get_named_field in favor of faster dict lookup (yah yah I'll show asv results in a bit)
git diff upstream/master -u -- "*.py" | flake8 --diff