-
Notifications
You must be signed in to change notification settings - Fork 1
feat: migrate wasi_nn #31
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is neededmigrationWasmEdge/WasmEdge#3840 plugin migrationWasmEdge/WasmEdge#3840 plugin migration
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is neededmigrationWasmEdge/WasmEdge#3840 plugin migrationWasmEdge/WasmEdge#3840 plugin migration
Type
Fields
Give feedbackNo fields configured for issues without a type.
Part of the C++ plugin migration (upstream: WasmEdge/WasmEdge#3840). Blocked by the pilot migration (see Relationships) — follow the recipe in
docs/MIGRATING.mdonce it lands.Migrate
wasi_nnto build, test, and ship from this repository.wasi_nnin the standalone CMake build (its existingWASMEDGE_PLUGIN_*option) against the pinned WasmEdge SDKtest/plugins/wasi_nnand make them pass locally — or add a smoke test if no upstream suite exists.github/workflows/build-wasi_nn.yml+.github/scripts/wasi_nn/wasi_nn: tick it off in the core-removal tracking issueMigration notes: largest plugin: one build/CI flavor per backend (GGML, PyTorch, TensorFlow Lite, OpenVINO, whisper, …) — split backends into checklist items when starting; PR #3 already prototypes the GGML workflow.
Known defect to fix when enabling the imported helper scripts:
utils/wasi-nn/build-wasinn-ubuntu-openvino.sh,utils/wasi-nn/install-openvino.sh, andutils/wasi-nn/install-openvino-genai.shuse[[ ! -v "${VAR}" ]](7 occurrences) —[[ -v ]]expects a variable name, not its value, so set variables can still be clobbered by defaults. Replace with[[ ! -v VAR ]]or: "${VAR:=default}". Found in review: #44 (comment)🤖 Generated by Claude Fable 5 with Claude Code