Skip to content

Make dask[distributed] optional via covalent[dask]- no dask install by default#2014

Closed
araghukas wants to merge 24 commits into
developfrom
ara/make-dask-optional
Closed

Make dask[distributed] optional via covalent[dask]- no dask install by default#2014
araghukas wants to merge 24 commits into
developfrom
ara/make-dask-optional

Conversation

@araghukas

@araghukas araghukas commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Require pip install covalent[dask] to install dask[distriburted] dependency for covalent, do not install dask[distributed] otherwise, with fallback to "local" executor.

Changes summary

Main point

  • Make dask[distributed] and optional dependency via covalent[dask] extra
  • Default executor changed from dask to local in defaults.py

Necessary fixes

  • Dropped Python 3.10 support, added Python 3.13; updated test_matrix.json and CI workflows accordingly
  • Fixed extract_graph to handle both "edges" (networkx ≥3.x) and "links" (older networkx) as the edge key; added lower bound on networkx version
  • Fixed various 422 errors by setting additional Content-Type: application/json headers as necessary
  • Moved locust out of tests/requirements.txt into the benchmark workflow only, to avoid a gevent conflict
  • Various test fixes: renamed statusjob_status, corrected mock paths, updated graph key references
  • I have added the tests to cover my changes.
  • I have updated the documentation and CHANGELOG accordingly.
  • I have read the CONTRIBUTING document.

@araghukas araghukas requested review from a team as code owners April 13, 2026 12:46
@araghukas araghukas changed the title Make 'dask[distributed]' optoinal via 'covalent[dask]', no dask install by default Make dask[distributed] optional via covalent[dask]- no dask install by default Apr 13, 2026
@araghukas

Copy link
Copy Markdown
Contributor Author

closing to try to bypass now-removed checks on Python 3.10

@araghukas araghukas closed this Apr 14, 2026
@araghukas araghukas reopened this Apr 14, 2026
@araghukas araghukas closed this Apr 14, 2026
@araghukas

Copy link
Copy Markdown
Contributor Author

replaced with: #2016

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