Skip to content

Conversation

michaelaye
Copy link
Contributor

The current docs would overwrite the cluster objects, so it needs to get its own variable name.
As the Client is usually named c, I changed the cluster object to the name cluster

Solves #4779

The current docs would overwrite the cluster objects, so it needs to get its own variable name.
As the Client is usually named `c`, I changed the cluster object to the name `cluster`
@martindurant
Copy link
Member

Fixes #4779 ( <- will automatically close your issue upon merger)

@michaelaye
Copy link
Contributor Author

Darn, I always confuse solve with resolve which also auto-closes.. ;)

@jrbourbeau
Copy link
Member

Fixes dask/dask#4779

@martindurant
Copy link
Member

Hah, I got it wrong too :)

@martindurant
Copy link
Member

I believe the failure here is due to dask/dask#4780

@martindurant martindurant merged commit e5d2488 into dask:master May 7, 2019
muammar added a commit to muammar/distributed that referenced this pull request May 8, 2019
* upstream/master:
  Add Type Attribute to TaskState (dask#2657)
  Add waiting task count to progress title bar (dask#2663)
  DOC: Clean up reference to cluster object (dask#2664)
  Allow scheduler to politely close workers as part of shutdown (dask#2651)
  Check direct_to_workers before using get_worker in Client (dask#2656)
  Fixed comment regarding keeping existing level if less verbose (dask#2655)
  Add idle timeout to scheduler (dask#2652)
  Avoid deprecation warnings (dask#2653)
  Use an LRU cache for deserialized functions (dask#2623)
  Rename Worker._close to Worker.close (dask#2650)
  Add Comm closed bookkeeping (dask#2648)
  Explain LocalCluster behavior in Client docstring (dask#2647)
  Add last worker into KilledWorker exception to help debug  (dask#2610)
  Set working worker class for dask-ssh (dask#2646)
  Add as_completed methods to docs (dask#2642)
  Add timeout to Client._reconnect (dask#2639)
  Limit test_spill_by_default memory, reenable it (dask#2633)
  Use proper address in worker -> nanny comms (dask#2640)
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.

3 participants