-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
CLN: some code cleanups #32177
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
CLN: some code cleanups #32177
Changes from all commits
f1f5dcd
84f0115
bc435ec
4f0c2eb
21f5020
1cc4e89
c78a6c7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -481,8 +481,7 @@ cdef class Interval(IntervalMixin): | |
|
||
@cython.wraparound(False) | ||
@cython.boundscheck(False) | ||
def intervals_to_interval_bounds(ndarray intervals, | ||
bint validate_closed=True): | ||
def intervals_to_interval_bounds(ndarray intervals, bint validate_closed=True): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. its worth double-checking me on this, but i think for non-cdef/cpdef functions, using cython syntax for the annotations doesnt help perf. in these cases, let's just use python versions There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. right, but that is orthogonal to this PR |
||
""" | ||
Parameters | ||
---------- | ||
|
Uh oh!
There was an error while loading. Please reload this page.