Skip to content

Add wait_for_workers flag in Dask Distributed config and executor (#131)#144

Merged
gpauloski merged 1 commit intomainfrom
issue-131
Sep 5, 2024
Merged

Add wait_for_workers flag in Dask Distributed config and executor (#131)#144
gpauloski merged 1 commit intomainfrom
issue-131

Conversation

@gpauloski
Copy link
Copy Markdown
Contributor

Description

Add wait_for_workers flag in Dask Distributed config and executor. This is helpful when the client is connected to a remote scheduler and you want to wait to submit work until the workers are connected to the scheduler.

Fixes

Type of Change

  • Bug (non-breaking change which fixes an issue)
  • Enhancement (non-breaking change which adds or improves functionality)
  • Internal (refactoring, performance, and testing)
  • Breaking (fix or feature that would cause existing functionality to not work as expected)
  • Documentation (no changes to the code)
  • Development (CI workflows, packages, templates, etc.)
  • Package (package dependencies and versions)

Testing

Added tests for coverage and tested with the cholesky app.

Pull Request Checklist

Please confirm the PR meets the following requirements.

  • Relevant tags are added (breaking, bug, documentation, enhancement, package, etc.).
  • Code changes pass pre-commit (e.g., ruff, mypy, etc.).
  • Tests have been added to show the fix is effective or that the new feature works.
  • New and existing unit tests pass locally with the changes.
  • Docs have been updated and reviewed if relevant.

@gpauloski gpauloski added the enhancement New features or improvements to existing functionality label Sep 5, 2024
@gpauloski gpauloski force-pushed the issue-131 branch 2 times, most recently from 7b07eca to 48c1a10 Compare September 5, 2024 21:50
@gpauloski gpauloski merged commit 455f303 into main Sep 5, 2024
@gpauloski gpauloski deleted the issue-131 branch September 5, 2024 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New features or improvements to existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add wait_for_workers flag to DaskExecutor

1 participant