Skip to content

Comments

Temporarily pin to pyodide-build==0.30.0, and ensure that the correct xbuildenvs are used#7788

Merged
stefanv merged 3 commits intoscikit-image:mainfrom
agriyakhetarpal:fix/temporarily-pin-pyodide-build
May 2, 2025
Merged

Temporarily pin to pyodide-build==0.30.0, and ensure that the correct xbuildenvs are used#7788
stefanv merged 3 commits intoscikit-image:mainfrom
agriyakhetarpal:fix/temporarily-pin-pyodide-build

Conversation

@agriyakhetarpal
Copy link
Contributor

@agriyakhetarpal agriyakhetarpal commented May 2, 2025

Description

Checklist

This PR serves as a temporary fix for #7783 to get CI green until we can get to the bottom of the issue. Also, another thing (non-breaking) that I addressed here is that we were using Pyodide 0.27.5 xbuildenvs at build time using pyodide build, and pyodide venv was downloading its own Pyodide xbuildenv/runtime (pinned via the PYODIDE_VERSION: environment variable) for testing packages. While this didn't impact anything, it could have been prone to errors when we would have released a new Pyodide version.

It would be great if #7440 could be merged sometime soon, though – it's ready for review/merging. It uses cibuildwheel to build and test the Pyodide/WASM wheels, and cibuildwheel pins the pyodide-build and Pyodide versions, so while that may make updating the testing configuration a little difficult, it's certainly going to much more stable.

cc: @stefanv

Release note

For maintainers and optionally contributors, please refer to the instructions on how to document this PR for the release notes.

...

@stefanv stefanv added the 🤖 type: Infrastructure CI, packaging, tools and automation label May 2, 2025
@stefanv stefanv merged commit 4dbc9fc into scikit-image:main May 2, 2025
25 of 26 checks passed
@stefanv
Copy link
Member

stefanv commented May 2, 2025

Thank you, @agriyakhetarpal!

@stefanv stefanv added this to the 0.26 milestone May 2, 2025
@agriyakhetarpal agriyakhetarpal deleted the fix/temporarily-pin-pyodide-build branch May 2, 2025 20:11
matthew-brett added a commit to matthew-brett/scikit-image that referenced this pull request Jul 7, 2025
* origin/main:
  Deprecate estimate method in favor of class constructor (scikit-image#7771)
  Temporary fix for Visual Studio & Clang incompatibility in Windows image (scikit-image#7835)
  Address deprecations in Pillow 11.3 (scikit-image#7828)
  Remove unused & obsolete `legacy_datasets`, `legacy_registry` vars (scikit-image#7677)
  Draft migration guide for skimage2 (scikit-image#7785)
  Do not report failure in wheels sub-recipe (scikit-image#7806)
  Use consistent wording in property description. (scikit-image#7804)
  Add intensity_median to regionprops (scikit-image#7745)
  CI: Update pypa/gh-action-pypi-publish to v1.12.4 for attestations on PyPI (scikit-image#7793)
  Document output dtype for transform.resize. (scikit-image#7792)
  Use `cibuildwheel` to build WASM/Pyodide wheels for `scikit-image`, push nightlies to Anaconda.org (scikit-image#7440)
  DOC: Include missing gain parameter in adjust_gamma equation (scikit-image#7763)
  Temporarily pin to `pyodide-build==0.30.0`, and ensure that the correct xbuildenvs are used (scikit-image#7788)
  Deprecate old names & attributes in RegionProperties (scikit-image#7778)
  Pin JasonEtco/create-an-issue action to SHA for v2.9.2 (scikit-image#7787)
  Make doctest-plus work with spin (scikit-image#7786)
  Report failures on main via issue (scikit-image#7752)
  Use `workers` instead of alternate parameter names (scikit-image#7302)
  Fix f-string in otsu plot (scikit-image#7780)
  Further document use of regionprops function and fix terms. (scikit-image#7518)
matthew-brett added a commit to matthew-brett/scikit-image that referenced this pull request Jul 14, 2025
* origin/main: (31 commits)
  Update import convention in certain gallery examples (scikit-image#7764)
  Refactor fundamental matrix scaling (scikit-image#7767)
  Add unit test for cval unequal to zero
  Forward  in _generic_edge_filter
  Remove superfluous mask argument from _generic_edge_filter
  Only report failure on main branch once
  Deprecate estimate method in favor of class constructor (scikit-image#7771)
  Temporary fix for Visual Studio & Clang incompatibility in Windows image (scikit-image#7835)
  Address deprecations in Pillow 11.3 (scikit-image#7828)
  Remove unused & obsolete `legacy_datasets`, `legacy_registry` vars (scikit-image#7677)
  Draft migration guide for skimage2 (scikit-image#7785)
  Do not report failure in wheels sub-recipe (scikit-image#7806)
  Use consistent wording in property description. (scikit-image#7804)
  Add intensity_median to regionprops (scikit-image#7745)
  CI: Update pypa/gh-action-pypi-publish to v1.12.4 for attestations on PyPI (scikit-image#7793)
  Document output dtype for transform.resize. (scikit-image#7792)
  Use `cibuildwheel` to build WASM/Pyodide wheels for `scikit-image`, push nightlies to Anaconda.org (scikit-image#7440)
  DOC: Include missing gain parameter in adjust_gamma equation (scikit-image#7763)
  Temporarily pin to `pyodide-build==0.30.0`, and ensure that the correct xbuildenvs are used (scikit-image#7788)
  Deprecate old names & attributes in RegionProperties (scikit-image#7778)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖 type: Infrastructure CI, packaging, tools and automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants