Skip to content

Migrate development and CI to uv and remove Upstream checks - #1352

Draft
Billyzhang1229 wants to merge 2 commits into
sgkit-dev:mainfrom
Billyzhang1229:uv-upgrade
Draft

Billyzhang1229 wants to merge 2 commits into
sgkit-dev:mainfrom
Billyzhang1229:uv-upgrade

Conversation

@Billyzhang1229

@Billyzhang1229 Billyzhang1229 commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #1351.

This PR moves development and CI dependency management to uv and removes the Upstream dependency checks. These changes are split into two commits.

Changes included:

  • Remove the Upstream workflow, its installation script, the README badge, and related contributor documentation.
  • Replace the requirements files with dependency groups in pyproject.toml and a committed uv.lock.
  • Provide test, docs, lint, and packaging groups, with an aggregate dev group for local development.
  • Update contributor instructions to use uv sync and uv run.
  • Use locked dependencies in standard CI, including scheduled runs, and document periodic, reviewed lockfile updates.
  • Keep cd docs && make html using the shared .venv. Docs workflows install dask==2026.8.0 for task-graph rendering with NumPy 2, and the Makefile uses uv run --no-sync to preserve that override. Document local setup and restoring locked dependencies before testing.
  • Migrate package builds and wheel installation checks to uv.
  • Retain Cubed and Zarr compatibility testing. Limit Cubed upgrades to the experimental packages and any necessary dependency changes.

Existing runtime dependency ranges, optional extras, and Python support are preserved.

Left for follow-up PRs:

  • Runtime dependency upgrades and relaxed version bounds, including newer Dask, xarray, numba and SciPy versions.
  • Address any compatibility changes those upgrades require, including a previously identified tajimas_D issue.
  • Remove the temporary documentation Dask override once a common version passes both tests and documentation builds.
  • Upgrading the bio2zarr/Zarr stack and broader Zarr 3 support.
  • Revisit the disabled GPU workflow.

Validation: local tests passed with 100% coverage; strict documentation builds and package smoke tests passed. Build, Windows, Check docs, and Cubed CI passed. The upstream-only Wheels workflow and GPU workflow were not exercised.

Delete the Upstream workflow and its installation script, and remove
the README badge and contributor documentation describing the workflow.

Fixes sgkit-dev#1351.
@Billyzhang1229 Billyzhang1229 changed the title Uv upgrade Migrate development and CI to uv and remove Upstream checks Sep 27, 2026
Replace requirements files with dependency groups and a committed uv.lock.
Use locked environments for standard CI and preserve existing runtime
dependency ranges and Python support.

Retain the documentation make target and use the shared .venv, with an
explicit Dask override for documentation builds. Document local setup and
restoration of locked dependencies, plus periodic reviewed lockfile updates.
Limit Cubed upgrades to its experimental packages.

This branch has not been deployed

No deployments
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.

delete upstream workflow

1 participant