-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
map_partitions with review comments #6776
Conversation
Thanks for keeping at it @kumarprabhu1988 - I think this is getting close. The missing piece now is docs. Could you add a section to the examples here: Line 574 in 9d19952
|
@jsignell Apologies for the delay. I've updated the docs. Let me know if this looks good. |
I really think the doc string should include an example using partition_info. Would you be able to add that? |
I'll merge this when it passes. |
Just merged |
Just checking in @jsignell, is this still good to go? |
* map_partitions with review comments * update doc * Add example to docstring Co-authored-by: Julia Signell <jsignell@gmail.com> Co-authored-by: James Bourbeau <jrbourbeau@gmail.com>
This PR adds functionality to return partition_info in a call to map_partitions. It is useful for distributed implementations of many algorithms. This PR includes changes for review comments in this PR: #6755.
black dask
/flake8 dask