Pin dask-ml for dask-expr compatibility #4
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@grovduck I'm not 100% sure we'll end up using
dask-ml
at all, but I figured I'd get this in place to avoid surprises down the line, since having the wrongdask-ml
breaks the Gist example.Details
There's an incompatibility between dask-ml<2024.3.20 and dask>=2024.3.0 due to the new dask-expr optimization. This pins dask-ml for backwards compatibility.
See dask/dask-ml#983
EDIT: CI is failing because there are no tests to run 🤦♂️