-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
feat(rust, python): Add null_on_oob
parameter to expr.array.get
#15426
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #15426 +/- ##
==========================================
- Coverage 81.13% 81.12% -0.01%
==========================================
Files 1362 1362
Lines 174812 174866 +54
Branches 2531 2531
==========================================
+ Hits 141829 141867 +38
- Misses 32498 32514 +16
Partials 485 485 ☔ View full report in Codecov by Sentry. |
@@ -40,3 +47,24 @@ pub fn array_get(ca: &ArrayChunked, index: &Int64Chunked) -> PolarsResult<Series | |||
), | |||
} | |||
} | |||
|
|||
#[inline] |
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 shouldn't be inlined.
Thank you @JamesCE2001 |
CodSpeed Performance ReportMerging #15426 will not alter performanceComparing Summary
|
…ola-rs#15426) Co-authored-by: James Edwards <[email protected]> Co-authored-by: Ritchie Vink <[email protected]>
Another part of #15240