Skip to content

Have local-cluster respect dask.config.num_workers#6058

Open
abergou wants to merge 1 commit intodask:mainfrom
abergou:local-cluster-cpu-count
Open

Have local-cluster respect dask.config.num_workers#6058
abergou wants to merge 1 commit intodask:mainfrom
abergou:local-cluster-cpu-count

Conversation

@abergou
Copy link
Copy Markdown

@abergou abergou commented Apr 1, 2022

Have local-cluster respect the num_workers configuration option if its specified in the config instead of taking over all CPUs on a machine by default.

  • Closes #xxxx
  • Tests added / passed
  • Passes pre-commit run --all-files

Have local-cluster respect the num_workers configuration option if its
specified in the config instead of taking over all CPUs on a machine by
default.
@GPUtester
Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2022

Unit Test Results

       18 files  +       6         18 suites  +6   9h 4m 40s ⏱️ + 3h 2m 11s
  2 697 tests +       3    2 613 ✔️ +     14       82 💤  -     9  2  - 2 
24 121 runs  +8 015  22 904 ✔️ +7 643  1 215 💤 +374  2  - 2 

For more details on these failures, see this check.

Results for commit 5d643ab. ± Comparison against base commit ccb0362.

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.

2 participants