Skip to content

Switch from dask to dask-core - #751

Merged
xylar merged 3 commits into
masterfrom
copilot/switch-from-dask-to-dask-core
Aug 20, 2026
Merged

Switch from dask to dask-core#751
xylar merged 3 commits into
masterfrom
copilot/switch-from-dask-to-dask-core

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The full dask package pulls in unnecessary extras (distributed, bokeh, etc.). dask-core provides the core functionality that MPAS-Tools actually needs.

Changes

  • dev-spec.txt, pixi.toml, recipe/recipe.yaml: Replace dask with dask-core (the lightweight conda package)
  • pyproject.toml: Retain dask — PyPI does not have a separate dask-core package; the split only exists in the conda ecosystem

Copilot AI linked an issue Aug 20, 2026 that may be closed by this pull request
Co-authored-by: xylar <4179064+xylar@users.noreply.github.com>
Copilot AI changed the title [WIP] Switch from dask to dask-core Switch from dask to dask-core Aug 20, 2026
Copilot AI requested a review from xylar August 20, 2026 13:49
@xylar

xylar commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

@copilot, it also appears that jinja2 is a missing requirement. This one needs to be added in all 3 modified files and also pyproject.toml.

pyyaml is a test-only dependency. Can you make sure it's taken care of where needed?

…ecipe.yaml

Co-authored-by: xylar <4179064+xylar@users.noreply.github.com>

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot, it also appears that jinja2 is a missing requirement. This one needs to be added in all 3 modified files and also pyproject.toml.

pyyaml is a test-only dependency. Can you make sure it'...

Done in the latest commit. jinja2 has been added as a runtime dependency in all 4 files (dev-spec.txt, pixi.toml, recipe/recipe.yaml, pyproject.toml). For pyyaml: it was already test-only in pyproject.toml (under [project.optional-dependencies] dev) and dev-spec.txt (under the # Development section). I've added it to the test requirements.run in recipe/recipe.yaml where it was missing.

@xylar
xylar marked this pull request as ready for review August 20, 2026 14:33

@xylar xylar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@xylar
xylar merged commit b394c80 into master Aug 20, 2026
5 checks passed
@xylar
xylar deleted the copilot/switch-from-dask-to-dask-core branch August 20, 2026 14:39
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.

Switch from dask to dask-core

2 participants