-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
HoTT TLM / Support VSpeak turbine protocol #3382
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
Conversation
CapnBry
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a turbine (but if someone want to send me one I would certainly destroy it same day) but this looks acceptable as a manufacturer supported extension.
I think you may want to do a pull for the docs repo that documents how turbines are supported over HoTT, so other manufacturers can replicate it easily? Just a short sentence about what brand this is and the 5 fields that are repurposed into RPM and temp
pkendall64
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments, take them with a grain of salt.
Yolo-ing this as I don't have a turbine, but the video is cool.
|
pkendall64
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noice!
This PR extends the HoTT TLM implementation to support VSpeak (https://www.vspeak-modell.de/en) turbine ECU telemetry providing turbine vitals: ECU voltage, Pump voltage or PWM depending on turbine type, RPM, Fuel remaining in ml, Fuel consumption in ml/min, EGT, Throttle, and status.
https://youtu.be/Wsp9HscGZ60?si=Ks8zLKUbPqulmS9T