Skip to content

CI: require minimum Dask version to avoid missing dask.dataframe.api - #348

Draft
rithiksachdeva wants to merge 1 commit into
geopandas:mainfrom
rithiksachdeva:fix/min-dask-dataframe-api-floor
Draft

CI: require minimum Dask version to avoid missing dask.dataframe.api#348
rithiksachdeva wants to merge 1 commit into
geopandas:mainfrom
rithiksachdeva:fix/min-dask-dataframe-api-floor

Conversation

@rithiksachdeva

Copy link
Copy Markdown

A closed CI run from PR #347 exposed an edge-case environment solve where an older Dask version was installed due to an unbounded dependency in a conda env YAML.

In that environment, test collection failed with:

ModuleNotFoundError: No module named 'dask.dataframe.api'

This occurs when the solver selects a Dask version that predates the introduction of dask.dataframe.api, which is required by the current codebase.

This change updates env YAML files that previously listed dask without a version constraint and adds a minimum compatible version floor.

The minimum version is chosen as the higher of:

  • the first Dask release that provides dask.dataframe.api
  • the existing package requirement in pyproject.toml:
    dask[dataframe]>=2025.1.0

This ensures CI environments cannot resolve to incompatible Dask versions while preserving flexibility in the "latest" environments across platforms.

This issue was platform/solver-specific (observed in macOS CI) and did not consistently reproduce across all environments.

@rithiksachdeva
rithiksachdeva deleted the fix/min-dask-dataframe-api-floor branch April 9, 2026 05:09
@rithiksachdeva
rithiksachdeva restored the fix/min-dask-dataframe-api-floor branch April 9, 2026 05:19
@rithiksachdeva rithiksachdeva reopened this Apr 9, 2026
@rithiksachdeva
rithiksachdeva force-pushed the fix/min-dask-dataframe-api-floor branch from 024000f to 4549d4c Compare April 9, 2026 19:45
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