Skip to content

update onnxruntime package name and hash for riscv64-spacemit #3481

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
csukuangfj merged 2 commits into from
Apr 8, 2026
Merged

update onnxruntime package name and hash for riscv64-spacemit #3481

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9e30ece
fix: update onnxruntime package name and hash for riscv64-spacemit
alex-spacemit Apr 8, 2026
40b0792
feat: add support for SpacemiT-K1 and SpacemiT-K3 platforms in README
alex-spacemit Apr 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions .github/workflows/riscv64-spacemit-linux.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -256,8 +256,6 @@ jobs:
export QEMU_LD_PREFIX=$GITHUB_WORKSPACE/toolchain/sysroot
export LD_LIBRARY_PATH=$GITHUB_WORKSPACE/toolchain/sysroot/lib
export QEMU_ARGS="-cpu max,vlen=256,elen=64,vext_spec=v1.0"
Comment on lines 256 to 258

Copilot AI Apr 8, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This removes the previously documented workaround (SPACEMIT_EP_DISABLE_OP_TYPE_FILTER) without leaving any trace of why it’s no longer needed. To reduce future churn when regressions happen, consider adding a brief comment tying the removal to the onnxruntime/EP version bump (e.g., “fixed in spacemit-ort 2.0.2+rc1”), or keep an opt-in toggle (only set the env var when a workflow input/variable indicates it) so the workaround can be re-enabled quickly if needed.

Copilot uses AI. Check for mistakes.
echo "Some mistakes in ep graph partition, disable op Gather for spacemit-ep now, will be fixed soon."
export SPACEMIT_EP_DISABLE_OP_TYPE_FILTER="Gather"

wget -q https://github.com/k2-fsa/sherpa-onnx/releases/download/asr-models/sherpa-onnx-streaming-zipformer-zh-14M-2023-02-23.tar.bz2
tar xvf sherpa-onnx-streaming-zipformer-zh-14M-2023-02-23.tar.bz2
Expand All @@ -280,8 +278,6 @@ jobs:
export QEMU_LD_PREFIX=$GITHUB_WORKSPACE/toolchain/sysroot
export LD_LIBRARY_PATH=$GITHUB_WORKSPACE/toolchain/sysroot/lib
export QEMU_ARGS="-cpu max,vlen=256,elen=64,vext_spec=v1.0"
Comment on lines 278 to 280

Copilot AI Apr 8, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This removes the previously documented workaround (SPACEMIT_EP_DISABLE_OP_TYPE_FILTER) without leaving any trace of why it’s no longer needed. To reduce future churn when regressions happen, consider adding a brief comment tying the removal to the onnxruntime/EP version bump (e.g., “fixed in spacemit-ort 2.0.2+rc1”), or keep an opt-in toggle (only set the env var when a workflow input/variable indicates it) so the workaround can be re-enabled quickly if needed.

Copilot uses AI. Check for mistakes.
echo "Some mistakes in ep graph partition, disable op Gather;Cast;ConvTranspose for spacemit-ep now, will be fixed soon."
export SPACEMIT_EP_DISABLE_OP_TYPE_FILTER="Gather;Cast;ConvTranspose"

wget -q https://github.com/k2-fsa/sherpa-onnx/releases/download/tts-models/vits-piper-en_US-lessac-medium.tar.bz2
tar xf vits-piper-en_US-lessac-medium.tar.bz2
Expand Down
4 changes: 4 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,8 @@ on the following platforms and operating systems:
- [旭日X3派][旭日X3派]
- [爱芯派][爱芯派]
- [RK3588][RK3588]
- [SpacemiT-K1][SpacemiT-K1]
- [SpacemiT-K3][SpacemiT-K3]
Comment on lines +94 to +95

Copilot AI Apr 8, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The README uses SpacemiT casing here, while the rest of the repo (e.g., filenames like riscv64-spacemit) uses spacemit. It’d be clearer to standardize casing across documentation and build/CI terminology (either adopt the brand casing everywhere in docs or use the repo’s existing Spacemit/Spacemit convention consistently).

Suggested change
- [SpacemiT-K1][SpacemiT-K1]
- [SpacemiT-K3][SpacemiT-K3]
- [Spacemit-K1][SpacemiT-K1]
- [Spacemit-K3][SpacemiT-K3]

Copilot uses AI. Check for mistakes.
- etc

with the following APIs
Expand Down Expand Up @@ -640,3 +642,5 @@ cross-platform Node.js microphone streaming library with prebuilt native binarie
[qnn-doc]: https://k2-fsa.github.io/sherpa/onnx/qnn/index.html
[ascend-doc]: https://k2-fsa.github.io/sherpa/onnx/ascend/index.html
[axera-npu]: https://axera-tech.com/Skill/166.html
[SpacemiT-K1]: https://cdn-resource.spacemit.com/file/chip/K1/K1_brief_zh.pdf
[SpacemiT-K3]: https://cdn-resource.spacemit.com/file/chip/K3/K3_brief_zh.pdf
Comment on lines +645 to +646

Copilot AI Apr 8, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The README uses SpacemiT casing here, while the rest of the repo (e.g., filenames like riscv64-spacemit) uses spacemit. It’d be clearer to standardize casing across documentation and build/CI terminology (either adopt the brand casing everywhere in docs or use the repo’s existing Spacemit/Spacemit convention consistently).

Suggested change
[SpacemiT-K1]: https://cdn-resource.spacemit.com/file/chip/K1/K1_brief_zh.pdf
[SpacemiT-K3]: https://cdn-resource.spacemit.com/file/chip/K3/K3_brief_zh.pdf
[Spacemit-K1]: https://cdn-resource.spacemit.com/file/chip/K1/K1_brief_zh.pdf
[Spacemit-K3]: https://cdn-resource.spacemit.com/file/chip/K3/K3_brief_zh.pdf

Copilot uses AI. Check for mistakes.
4 changes: 2 additions & 2 deletions cmake/onnxruntime-linux-riscv64-spacemit.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ if(NOT BUILD_SHARED_LIBS)
message(FATAL_ERROR "This file is for building shared libraries. BUILD_SHARED_LIBS: ${BUILD_SHARED_LIBS}, SHERPA_ONNX_ENABLE_SPACEMIT: ${SHERPA_ONNX_ENABLE_SPACEMIT}")
endif()

set(onnxruntime_pkg_name "spacemit-ort.riscv64.2.0.1.tar.gz")
set(onnxruntime_pkg_name "spacemit-ort.riscv64.2.0.2+rc1.tar.gz")
set(onnxruntime_URL "https://archive.spacemit.com/spacemit-ai/onnxruntime/${onnxruntime_pkg_name}")

Copilot AI Apr 8, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+ in the tarball filename can be mishandled by some URL/HTTP tooling (it is sometimes decoded as a space by non-compliant components). To make downloads more robust, consider either (a) URL-encoding the + in the URL (use %2B in the computed download URL) or (b) publishing/using an equivalent filename without + (e.g., -rc1) and referencing that instead.

Suggested change
set(onnxruntime_URL "https://archive.spacemit.com/spacemit-ai/onnxruntime/${onnxruntime_pkg_name}")
string(REPLACE "+" "%2B" onnxruntime_pkg_name_url "${onnxruntime_pkg_name}")
set(onnxruntime_URL "https://archive.spacemit.com/spacemit-ai/onnxruntime/${onnxruntime_pkg_name_url}")

Copilot uses AI. Check for mistakes.
set(onnxruntime_HASH "SHA256=8a15035aca34d5fd95f24444d4c7843265c1a81f49d84ec6fe9c6d0fdf5b55cf")
set(onnxruntime_HASH "SHA256=36276ce948da5cb9e388f1efcbc24849f1a10dd6710d0b02943bd2fe54bad87f")

# If you don't have access to the Internet,
# please download onnxruntime to one of the following locations.
Expand Down
Loading