-
Notifications
You must be signed in to change notification settings - Fork 927
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
Support args=
in apply
#9514
Support args=
in apply
#9514
Conversation
Codecov Report
@@ Coverage Diff @@
## branch-21.12 #9514 +/- ##
================================================
- Coverage 10.79% 10.64% -0.15%
================================================
Files 116 117 +1
Lines 18869 19342 +473
================================================
+ Hits 2036 2059 +23
- Misses 16833 17283 +450
Continue to review full report at Codecov.
|
rerun tests |
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.
It's great that numba makes it easy to handle this so elegantly.
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.
LGTM!
@gpucibot merge |
Opened #9598 for the |
Closes #9500
Allows passing
args=
toDataFrame.apply
as is supported in pandas: https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.apply.htmlConcretely, this allows for this:
cc @randerzander