-
Notifications
You must be signed in to change notification settings - Fork 16.9k
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
Rover: Notify lat and long to the seventh decimal place #27020
Rover: Notify lat and long to the seventh decimal place #27020
Conversation
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.
Fit in a single message anyway. LGTM
7cb12ea
to
3ade1bd
Compare
Does it actually display all the digits? I suspect it does in which case I should do the same in the recent slung payload simulator. |
I haven't provoked it to do so, but we may as well give it every chance. |
@@ -33,7 +33,7 @@ bool Rover::set_home(const Location& loc, bool lock) | |||
mode_auto.mission.write_home_to_storage(); | |||
|
|||
// send text of home position to ground stations | |||
gcs().send_text(MAV_SEVERITY_INFO, "Set HOME to %.6f %.6f at %.2fm", | |||
gcs().send_text(MAV_SEVERITY_INFO, "Set HOME to %.7f %.7f at %.2fm", |
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.
this won't work on ChibiOS at least, possibly not on SITL too, we go via a float inside the printf engine
We think this doesn't actually work. Remember to provide testing results! We discussed on the dev call that what we need is a function somewhere that allows printing lat,lon values to the required number of decimal places |
The seventh decimal place corresponds to centimeters.