webonnx / wonnx

A WebGPU-accelerated ONNX inference run-time written 100% in Rust, ready for native and the web
Other
1.61k stars 59 forks source link

set async_recursion to ?Send for wasm32 arch #204

Closed astnmsn closed 6 months ago

astnmsn commented 6 months ago

In the dependency upgrade for wgpu from 16 -> 19, wbgpu started properly labeling their structs as non send/sync which broke the expectations for the async_recursion proc macro. By configuring the macro to not expect a type that is Send, the wasm build of wgpu should properly compile. No callsites of the updaded function require that the type be Send, so there wasn't a need to update anything else.

pixelspark commented 6 months ago

Thanks for sorting this out @astnmsn 💯