Skip to content

nagisa: Add version 0.3.1 - #4769

Open
luhenry wants to merge 4 commits into
mainfrom
port/nagisa
Open

luhenry wants to merge 4 commits into
mainfrom
port/nagisa

Conversation

@luhenry

@luhenry luhenry commented Nov 9, 2026 •

Copy link
Copy Markdown
Member

Compiles nagisa_utils, a Cython extension backing a Japanese word segmentation/POS tagger. Upstream publishes no riscv64 wheel.

Mirrors upstream's own build_wheels.yml ([tool.cibuildwheel] in pyproject.toml carries the build options).

Differs from upstream

  • Drops musllinux - DyNet38, an install-time dependency for python < 3.15, has no musl riscv64 wheel.

Matrix: cp312/cp313/cp314 only, hard-coded rather than the dynamic per-interpreter matrix - DyNet38 has riscv64 wheels for exactly those three interpreters (no cp310, cp311, cp314t, cp315(t)), and python-exclude can't filter by platform to also drop musllinux.

Testing

  • Same as upstream's test/check_wheel.py smoke test.

License: OK

Built on cp312/cp313/cp314; wheel installs with DyNet38 resolved from this registry and tags/segments the sample sentence as expected.

luhenry added a commit that referenced this pull request Nov 9, 2026
_publish-wheel.yml now requires a matrix input (main's "publish:
Validate effective wheel targets"), but this PR branch forked before
that landed and nagisa has never been merged to main, so
build-nagisa.yml never picked up the matching one-line update every
already-published package's workflow got. The merge ref combines the
new required-input _publish-wheel.yml with this branch's old
build-nagisa.yml, so any run has failed startup with "Input matrix is
required, but not provided".

Pass it the same way the mass-update commit did for every other
package's publish job.

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.

1 participant