search

intel / openvino-rs

Rust bindings for OpenVINO™
Apache License 2.0
80 stars 23 forks source link

Enable variadic args in dynamic linking #113

Open rahulchaphalkar opened 3 months ago

rahulchaphalkar commented 3 months ago

Added support for variadic args in dynamic link. Runtime linking is pending.

rahulchaphalkar commented 3 months ago

Do we still need this after #112?

I think its good to include this, then later on we only need to work on runtime linking. This works fine with workaround in https://github.com/intel/openvino-rs/pull/112 as well.

abrown commented 3 months ago

What I mean is, everything appears to work fine without this change, so why do we need it?

rahulchaphalkar commented 3 months ago

I added this as we will need this patch in the future if/when we fix runtime linking issue. Its fine to not include this at this time, in that case I will close/draft this PR.