search

intel / openvino-rs

Rust bindings for OpenVINO™
Apache License 2.0
84 stars 24 forks source link

Feature/send and sync #48

Closed geekbeast closed 2 years ago

geekbeast commented 2 years ago

This adds the necessary compiler promises to allow use of openvino backend in Viceroy and other wasmtime derived platforms.

geekbeast commented 2 years ago

Most of the classes seem Send as there isn't any thread local magic going on. CNNNetwork seems Sync unless binding to a session somehow mutates the network.

abrown commented 2 years ago

The CI failure you observed is not due to your changes; I opened and merged #49 to avoid that and if you rebase on main then CI should pass. On the changes themselves, the review is in progress: I'm checking internally with the OpenVINO team about this just to make sure this will be a safe change.