-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
STY: Boolean values for bint variables #33005
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
STY: Boolean values for bint variables #33005
Conversation
@@ -78,7 +78,7 @@ def inner_join(const int64_t[:] left, const int64_t[:] right, | |||
|
|||
@cython.boundscheck(False) | |||
def left_outer_join(const int64_t[:] left, const int64_t[:] right, | |||
Py_ssize_t max_groups, sort=True): | |||
Py_ssize_t max_groups, bint sort=True): |
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 the only place where I'm adding bint
I am not sure asv is always appropriate for changes that might only give a small change in performance. Can you maybe check the generated code to ensure that no more work is being done (unless this is documented in the cython docs to work fine?) |
@jorisvandenbossche I have checked the sources of the generated C-files. there is no code change at all (except for the comments displaying the equivalent code at the |
Super, thanks for checking! |
Hah, I just made the same suggestion in the next PR |
Thanks @MomIsBestFriend |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff
asv
benchmarks: to show that performance is not changed with the stylistic changes)