-
-
Notifications
You must be signed in to change notification settings - Fork 803
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
#3505: add fuel quantity sensor in telemetry simulator #3506
Conversation
It is helpful if you add text |
I would prefer if you put the new sensor below the existing |
@@ -158,6 +158,7 @@ void TelemetrySimulator::showEvent(QShowEvent *event) | |||
SET_INSTANCE(t2_inst, T2_FIRST_ID, 0); | |||
SET_INSTANCE(rpm_inst, RPM_FIRST_ID, DATA_ID_RPM); | |||
SET_INSTANCE(fuel_inst, FUEL_FIRST_ID, 0); | |||
SET_INSTANCE(fuel_qty_inst, FUEL_QTY_FIRST_ID, 0); |
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.
Pleas try to keep the alignment, if needed move the tabs on all other lines.
Closes #3505 |
Expect for minor issues (comments above) the code works good and will be merged as soon as comments are addressed. Thanks! |
@projectkk2glider I made the changes, added/updated code from #3507, pushed, but it seems github is rather slow to process new commits sometimes... |
Thanks! The code ported to |
It seemed to be missing. I have still no clue why this sensor is supposed to be sent as 0.01ml. Is there any sensor in this world capable to measure that? Should it be changed as well to something more reasonable? (ml is just enough, IMHO)