Skip to content
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

Client scatter docs #2320

Merged
merged 3 commits into from Oct 26, 2018
Merged

Client scatter docs #2320

merged 3 commits into from Oct 26, 2018

Conversation

@ericmjl
Copy link
Contributor

@ericmjl ericmjl commented Oct 26, 2018

@mrocklin, this is my (unsolicited) PR to the docs for client.scatter. I think I formed a misunderstanding of the client.scatter class method because I didn't have a simple example in the docs on how to use scatter with client.submit.

This PR hopefully rectifies that situation for many others who might encounter the same issue. I have done my best to make clear that the example I added is about using the scattered data in a parallelized function.

Based off your SO reply to my question: https://stackoverflow.com/questions/52997229/is-there-an-advantage-to-pre-scattering-data-objects-in-dask/52997326#52997326

ericmjl added 3 commits Oct 26, 2018
Merge from dask/master
@mrocklin
Copy link
Member

@mrocklin mrocklin commented Oct 26, 2018

Looks good. The linting error is due to a flake8 change upstream. Merging.

Thanks @ericmjl !

@mrocklin mrocklin merged commit 9afb07a into dask:master Oct 26, 2018
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@ericmjl ericmjl deleted the ericmjl:client-scatter-docs branch Oct 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants