Skip to content

pot: add build-pot.yml for riscv64 wheels - #1823

Merged
luhenry merged 2 commits into
mainfrom
pot
Sep 14, 2026
Merged

luhenry merged 2 commits into
mainfrom
pot

Conversation

@luhenry

@luhenry luhenry commented Sep 12, 2026

Copy link
Copy Markdown
Member

Compiles ot.lp.emd_wrap (network simplex), ot.partial.partial_cython, and the Eigen-backed ot.bsp.bsp_wrap, all OpenMP-parallel. Upstream publishes no riscv64 wheel.

Mirrors upstream's build_wheels.yml.

Differs from upstream

  • Only manylinux_riscv64 - upstream also builds macOS/Windows/other Linux arches and musllinux, none of which apply here.

Testing

  • Runs test/ only, not ot/, to avoid shadowing the compiled extension with the uncompiled checkout.

License: Wheel bundles Eigen (MPL2, plus a BSD-licensed AlignedBox::transform()); upstream ships no licence text for it, so the build adds it.

Patches

  • 0001-add-the-vendored-eigen-licences-next-to-the-package.patch - To upstream [same gap exists in every POT wheel on PyPI, needs a maintainer discussion]. Without it the wheel ships only POT's own MIT LICENSE despite compiling Eigen headers in; reproduces on any platform, not riscv64-only.

Built on cp312; pending CI results.

luhenry added a commit that referenced this pull request Sep 12, 2026
@luhenry
luhenry force-pushed the pot branch 2 times, most recently from d807326 to 6ff5f87 Compare September 12, 2026 12:10
Job logs (fetched via gh api .../jobs/103617883952/logs) show cp314t was
still passing tests (test/unbalanced/test_mm.py at 93%) when GitHub
cancelled the job at exactly 60m -- a genuine timeout, not a hang or a
real test failure. Free-threaded cp314t runs the suite noticeably
slower than cp312/313/314 (which finish in 45-53m), so it doesn't clear
the 60m job budget.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Az13NcXsVZzzxXmaxxUEy7
@luhenry
luhenry merged commit f8a70dd into main Sep 14, 2026
10 checks passed
@luhenry
luhenry deleted the pot branch September 14, 2026 09:28
@luhenry luhenry mentioned this pull request Sep 14, 2026
@luhenry luhenry linked an issue Sep 14, 2026 that may be closed by this pull request
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.

pot riscv64 support

1 participant