Enforce waypoint altitude tolerance fix

Open breadoven opened this issue 3 years ago • 3 comments

Closes https://github.com/iNavFlight/inav/issues/8278.

Not yet flight tested.

Aug 01 '22 15:08 breadoven

Nice! Will HITL test as soon as build check is finished. I hope Pawel accepts it for 5.1 as it changes the config (compatibility)

Aug 01 '22 15:08 b14ckyy

Could always just change the current fixed 1m tolerance to perhaps 5m for 5.1 and keep this change for 6.0. Won't affect config compatibility in that case ?

Aug 01 '22 20:08 breadoven

5m should be fine. if that still is not enough then the tune is messed up anyway. I have a feeling that this barely affects multirotor anyway as they should fly to the WP in a linear way? or is that still optional?

Aug 01 '22 21:08 b14ckyy

@breadoven @b14ckyy can we merge?

Aug 15 '22 12:08 DzikuVx

@breadoven @b14ckyy can we merge?

Simulator and real flight tested. Has a go from my side.

Aug 15 '22 12:08 b14ckyy