Skip to content

Small fix for executor example in README.md#11

Open
dee-me-tree-or-love wants to merge 1 commit intohammerlab:masterfrom
dee-me-tree-or-love:patch-1
Open

Small fix for executor example in README.md#11
dee-me-tree-or-love wants to merge 1 commit intohammerlab:masterfrom
dee-me-tree-or-love:patch-1

Conversation

@dee-me-tree-or-love
Copy link

In the README example, there was a problem with using dask.delayed since delayed is imported from dask:

from math import sqrt
from dask.distributed import Executor
from dask import delayed
# no import of `dask`
...
tasks = [dask.delayed(sqrt)(i) for i in range(100)] 
# `dask` is not declared

tasks = [delayed(sqrt)(i) for i in range(100)]
# works
...

I know that's just a little picky note, but I hope it maybe helps ✌️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant