-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
ARROW-12911: [Python] Export scalar aggregate options to pc.sum #10433
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.
Thanks, looks good.
There is a sum
method on the Array object as well. Maybe we can pass through keyword arguments there as well?
@@ -836,12 +836,6 @@ cdef class Array(_PandasConvertible): | |||
result = GetResultValue(self.ap.View(type.sp_type)) | |||
return pyarrow_wrap_array(result) | |||
|
|||
def sum(self): |
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.
Remove this sum
?
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.
Actually removing this method is a backwards incompatible change. If we want to do that, I think we should deprecate it first (although it is certainly strange to only have a single reduction method exposed here)
For now I would maybe just pass through **kwargs
(or leave as as)
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.
Added kwargs to array.sum(). Tested array.sum(min_count=0).
Thanks! |
Closes apache#10433 from cyb70289/12911-py-sum Authored-by: Yibo Cai <yibo.cai@arm.com> Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
No description provided.