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

Rover: SITL RTL_SPEED only allows 1m/s increments contrary to description #26995

Closed
Kraligor opened this issue May 6, 2024 · 1 comment · Fixed by #27002
Closed

Rover: SITL RTL_SPEED only allows 1m/s increments contrary to description #26995

Kraligor opened this issue May 6, 2024 · 1 comment · Fixed by #27002

Comments

@Kraligor
Copy link

Kraligor commented May 6, 2024

Bug report

Issue details

Setting RTL_SPEED <1 will result in RTL_SPEED not being used at all. Setting RTL_SPEED to any float will result in the decimal places being ignored and only the integer-part being used (e.g. 0.9 will result in 0, 4.6 will result in 4).

This is contradictory to the description in Parameters.cpp:

// @Units: m/s
// @Range: 0 100
// @Increment: 0.1

Version
ArduRover V4.6.0-dev

Platform
[ ] All
[ ] AntennaTracker
[ ] Copter
[ ] Plane
[x] Rover
[ ] Submarine

Airframe type
SITL
Hardware type
SITL

Logs
https://discuss.ardupilot.org/uploads/short-url/xgXJUfS4ldgNkxtRu4D4jsJzZ42.tlog
https://discuss.ardupilot.org/uploads/short-url/gksP74ZXbpDt6plsYPHjsrjEkbM.param

Associated forum thread: https://discuss.ardupilot.org/t/rover-rtl-speed-issue/117281

@rmackay9 rmackay9 changed the title ArduRover SITL RTL_SPEED only allows 1m/s increments contrary to description Rover: SITL RTL_SPEED only allows 1m/s increments contrary to description May 6, 2024
@rmackay9
Copy link
Contributor

rmackay9 commented May 6, 2024

I've reproduced this issue in master and Rover-4.4 so this is really a bug. This is very surprising and I don't immediately see the cause but clearly something is going wrong.

Thanks for finding this.

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

Successfully merging a pull request may close this issue.

2 participants