Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Roadmap for higher-level methods? #5

Closed
quietlychris opened this issue Aug 26, 2023 · 4 comments
Closed

Roadmap for higher-level methods? #5

quietlychris opened this issue Aug 26, 2023 · 4 comments

Comments

@quietlychris
Copy link

Hi there! Awesome to see that BR folks have spent some time working on making Rust implementations of libraries for the acoustics hardware (also, very much appreciate your work on mavlink as well).

I was just wondering if there were plans for continuing developing this library, and if so, if that was likely to include higher-level methods like the data = myPing.get_distance() kind of API that the Python version has available. Thanks!

@patrickelectric
Copy link
Member

Hi @quietlychris, thanks for your words.

Right now there is no time for me to work on that, was my personal development of the library based on my knowledge. I consider a proof of concept and a case of study for software architecture and future projects.

As you can see in #3, there is a PR for deserialization, and it's I believe that last thing necessary before pushing for high level sensor functions, that I could create once we finish it.

You are more than welcomed to collaborate with the library and continue @joaoantoniocardoso work in #3.

@quietlychris
Copy link
Author

Thanks for the additional context! I can't make any promises, but I'm definitely interested and will take a look at the work in #3 to see where things stand and if I might be able to anything to push it forwards a little :)

@RaulTrombin
Copy link
Member

Hi @quietlychris,
Please take a look at the new upgrades.

We aim to have the first official release soon.

Here is the updated methods list per device type:
Ping1D
Ping360

@quietlychris
Copy link
Author

@RaulTrombin This is exactly what I was hoping for! I pulled this as a dependency from crates.io and everything seemed to work as expected. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants