From 10893887820e2d392541d543fe0e0537b7be1e75 Mon Sep 17 00:00:00 2001 From: Ludovic Henry Date: Mon, 7 Sep 2026 19:34:30 +0200 Subject: [PATCH 1/2] dracopy: add build-dracopy.yml for riscv64 wheels Cython wrapper around Google's Draco mesh compression C++ library. Draco is vendored as a git submodule and statically linked (libdraco.a) into the DracoPy extension via legacy scikit-build. --- .github/workflows/build-dracopy.yml | 109 ++++++++++++++++++++++++++++ 1 file changed, 109 insertions(+) create mode 100644 .github/workflows/build-dracopy.yml diff --git a/.github/workflows/build-dracopy.yml b/.github/workflows/build-dracopy.yml new file mode 100644 index 000000000..eaea777b9 --- /dev/null +++ b/.github/workflows/build-dracopy.yml @@ -0,0 +1,109 @@ +# SPDX-FileCopyrightText: 2026 The RISE Project +# SPDX-License-Identifier: MIT +--- +# This workflow is based on the `build_wheels` job of +# https://github.com/seung-lab/DracoPy/blob/2.0.0/.github/workflows/build_aarch64.yml +name: Build dracopy wheels (riscv64) + +on: + workflow_dispatch: + inputs: + version: + description: 'DracoPy version to build (git tag equals the version, e.g. 2.0.0)' + required: true + default: '2.0.0' + pull_request: + paths: + - '.github/workflows/build-dracopy.yml' + +concurrency: + group: ${{ github.workflow }}-${{ inputs.version || '2.0.0' }}-${{ github.head_ref || github.run_id }} + cancel-in-progress: true + +permissions: + contents: read # to fetch code (actions/checkout) + +env: + # `inputs.version` is empty on pull_request events; default to 2.0.0 there. + DRACOPY_VERSION: ${{ inputs.version || '2.0.0' }} + MANYLINUX_RISCV64_IMAGE: quay.io/pypa/manylinux_2_39_riscv64 + +jobs: + setup: + uses: $/.github/workflows/_setup.yml + + build_wheels: + needs: [setup] + name: Build dracopy ${{ inputs.version || '2.0.0' }} ${{ matrix.python }}-manylinux_riscv64 + runs-on: ubuntu-24.04-riscv + timeout-minutes: 90 + strategy: + fail-fast: false + matrix: + python: + - "cp312" + - "cp313" + - "cp314" + - "cp314t" + + steps: + - name: Checkout DracoPy ${{ env.DRACOPY_VERSION }} + uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 + with: + repository: seung-lab/DracoPy + ref: ${{ env.DRACOPY_VERSION }} + # Google's Draco mesh compression library is a submodule and is + # statically linked (libdraco.a) into the DracoPy extension. + submodules: recursive + persist-credentials: false + + - name: Build wheels + uses: pypa/cibuildwheel@1828c10ab37f080699c7b81cea34097c684a7074 # v4.2.0 + with: + output-dir: wheelhouse/ + only: ${{ matrix.python }}-manylinux_riscv64 + env: + CIBW_MANYLINUX_RISCV64_IMAGE: ${{ env.MANYLINUX_RISCV64_IMAGE }} + # Mirrors upstream's own CIBW_BEFORE_BUILD (build_aarch64.yml); the + # submodule is already present via `submodules: recursive` above, so + # the `git submodule` calls upstream runs here are dropped. cmake and + # cython are setup.py setup_requires, not pyproject.toml build-system + # requires, so they need to already be importable/on PATH before + # setup.py runs, isolated build or not. + CIBW_BEFORE_BUILD: pip install "cmake>=3.30.2" cython numpy scikit-build setuptools wheel + # numpy and cmake have riscv64 wheels only on our registry. + CIBW_ENVIRONMENT: PIP_EXTRA_INDEX_URL=https://pypi.riseproject.dev/simple/ + CIBW_TEST_REQUIRES: pytest + CIBW_TEST_SOURCES: tests.py pytest.ini testdata_files + CIBW_TEST_COMMAND: cd {project} && python -m pytest -v tests.py + # numpy has no riscv64 wheel on public PyPI; keep pip on our registry's copy. + CIBW_TEST_ENVIRONMENT: PIP_ONLY_BINARY=numpy + + - name: Check wheel contents + run: | + python3 - wheelhouse/*.whl <<'EOF' + import sys, zipfile + path = sys.argv[1] + names = zipfile.ZipFile(path).namelist() + exts = [n for n in names if n.endswith(".so")] + assert exts and all("DracoPy" in n for n in exts), exts + licences = {n.split(".dist-info/licenses/", 1)[1] for n in names + if ".dist-info/licenses/" in n and not n.endswith("/")} + assert licences == {"LICENSE"}, licences + EOF + + - uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1 + with: + name: dracopy-${{ env.DRACOPY_VERSION }}-${{ matrix.python }}-manylinux_riscv64 + path: wheelhouse/*.whl + if-no-files-found: error + + publish: + name: Publish dracopy ${{ inputs.version || '2.0.0' }} + needs: [setup, build_wheels] + permissions: + contents: write + pull-requests: write + uses: $/.github/workflows/_publish-wheel.yml + with: + artifact-pattern: dracopy-${{ inputs.version || '2.0.0' }}-*-manylinux_riscv64 From cae21f76852286660c6600d0cf64e5c3e0509b80 Mon Sep 17 00:00:00 2001 From: Ludovic Henry Date: Mon, 7 Sep 2026 20:11:48 +0200 Subject: [PATCH 2/2] build-dracopy.yml: install numpy for the test phase tests.py imports numpy at module scope; CIBW_TEST_REQUIRES only listed pytest, so pytest test collection failed with ModuleNotFoundError. --- .github/workflows/build-dracopy.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build-dracopy.yml b/.github/workflows/build-dracopy.yml index eaea777b9..079255087 100644 --- a/.github/workflows/build-dracopy.yml +++ b/.github/workflows/build-dracopy.yml @@ -73,7 +73,7 @@ jobs: CIBW_BEFORE_BUILD: pip install "cmake>=3.30.2" cython numpy scikit-build setuptools wheel # numpy and cmake have riscv64 wheels only on our registry. CIBW_ENVIRONMENT: PIP_EXTRA_INDEX_URL=https://pypi.riseproject.dev/simple/ - CIBW_TEST_REQUIRES: pytest + CIBW_TEST_REQUIRES: pytest numpy CIBW_TEST_SOURCES: tests.py pytest.ini testdata_files CIBW_TEST_COMMAND: cd {project} && python -m pytest -v tests.py # numpy has no riscv64 wheel on public PyPI; keep pip on our registry's copy.