wasm-micro-runtime icon indicating copy to clipboard operation
wasm-micro-runtime copied to clipboard

wasi-nn: make `wasi_ephemeral_nn` default

Open yamt opened this issue 7 months ago • 2 comments

we use wasi_nn module for wasi-nn imports by default. i'd suggest to flip the default to use wasi_ephemeral_nn. (and probably deprecate wasi_nn and remove it in future.)

cf. https://github.com/bytecodealliance/wasm-micro-runtime/pull/1521#discussion_r2114910503

yamt avatar Jun 04 '25 02:06 yamt

Agree.

FYI, @HongxiaWangSSSS is targeting the same goal. @HongxiaWangSSSS , would you mind sharing the status of the second PR?

lum1n0us avatar Jun 04 '25 05:06 lum1n0us

(and probably deprecate wasi_nn and remove it in future.)

wasi_ephemeral_nn support was introduced by:

commit db025e457afd3846815e34b385ecd79bc3262f8d
Author: liang.he <[email protected]>
Date:   Mon Jun 17 14:58:09 2024 +0800

and has always been recommended in our documentation.

wrt releases, it has been included since WAMR-2.1.1 (2024-07)

i wonder if anyone is still using "wasi_nn".

yamt avatar Jun 17 '25 07:06 yamt

Agree.

FYI, @HongxiaWangSSSS is targeting the same goal. @HongxiaWangSSSS , would you mind sharing the status of the second PR?

its ok for me! wasi_ephemeral_nn seems to have worked.

HongxiaWangSSSS avatar Jun 19 '25 06:06 HongxiaWangSSSS

wasi_ephemeral_nn is now the default. build-time deprecation warning has been added too.

yamt avatar Aug 12 '25 07:08 yamt