Fusion icon indicating copy to clipboard operation
Fusion copied to clipboard

Rust bindings

Open avsaase opened this issue 1 year ago • 2 comments

Hi, thanks for your work. I would like to use this algorithm in a Rust project. I considered porting the code but then I found this comment where you stated you'd prefer wrapping the C library. I don't know C all that well but I would like to give this a try. Would a Rust library be in scope for this repository?

avsaase avatar Dec 05 '23 08:12 avsaase

I've ported this library to Rust and just published it hour ago Please look at https://github.com/mark2b/imu-fusion-rs It is not binding, but Rust native implementation

mark2b avatar Dec 05 '23 08:12 mark2b

It sounds like @mark2b has what you need.

It is within the scope of this repository to provide a Rust library that wraps the C library, but we are unlikely to make progress on this until required by a project or client.

xioTechnologies avatar Dec 05 '23 23:12 xioTechnologies

HI @xioTechnologies, are you interested in a PR that adds Rust bindings? I gave it a try and it turned out to be quite easy.

avsaase avatar Jul 02 '24 13:07 avsaase

I would be interested in seeing the code but we would not merge external PRs.

xioTechnologies avatar Jul 03 '24 08:07 xioTechnologies

Why is that?

You can find the code in my fork here: https://github.com/avsaase/Fusion/tree/rust-bindings

avsaase avatar Jul 03 '24 08:07 avsaase

To maintain quality. Thank you for sharing.

xioTechnologies avatar Jul 03 '24 08:07 xioTechnologies

Note that the bindings are currently not tested and could break in potentially subtle ways when the the C library changes.

I'll set up a separate repo and continue the development there.

avsaase avatar Jul 03 '24 09:07 avsaase