braincore / pid-rs

A PID controller for Rust projects.
Apache License 2.0
92 stars 26 forks source link

Derive common traits for `Pid`; update to 2018 edition #5

Closed hannobraun closed 4 years ago

braincore commented 4 years ago

Thanks for the PR and sorry for the delay! I haven't been using rust recently, and hadn't ever added serde as an optional feature. Can you please update the README to include this functionality?

hannobraun commented 4 years ago

Done!

braincore commented 4 years ago

Thanks! Published as v2.1.0. Let me know if you want any more credit than the git history.

If you don't mind, how are you using this crate?

hannobraun commented 4 years ago

Thanks for merging and publishing, and thanks for the library, of course! Credit in Git history is fine, thanks for offering.

If you don't mind, how are you using this crate?

I'm working on a multiplayer game (as a hobby) involving spaceships, and I'm using the PID controller for missile guidance (well, right now it's just a "P" controller; need to do some tuning :-) ).

I plan to publish the repository some time within the next few weeks. Do you want me to ping you once that happens?

braincore commented 4 years ago

Sure!

hannobraun commented 4 years ago

Took me a bit longer than expected, but it's out: https://github.com/hannobraun/vndf

Never quite got around to tuning the PID controller, unfortunately.