Inav: Failsafe procedure SET-THR not taking on Failsafe Throttle value if ALT HOLD enabled.
Firmware version 1.8. Hardware SPRacing F3 clone. Test platform quad 250. Failsafe procedure is SET-THR (formerly LAND)
Test condition 1- POS HOLD and ALT HOLD then force failsafe with tx OFF.
Test condition 2- POS HOLD then force failsafe with tx OFF.
(position hold only used to keep craft near)
Test condition 1 appears to hold the current throttle setting with no change in altitude after turning tx off then craft falls to the ground after the default 20 seconds.
Test condition 2 appears to work as expected with the craft descending briefly after turning tx off and then eventually powering off after the default 20 seconds.
Not yet confirmed by anyone else at this stage but repeated several times by myself.
KiwiDavid
All 12 comments
Duplicate of https://github.com/iNavFlight/inav/issues/1872
We somehow didn't merge the fix for the bug https://github.com/iNavFlight/inav/pull/1873
digitalentity
on 21 Nov 2017
@digitalentity This issue did spark an idea. If the failsafe mode is LAND/SET-THR, wouldn't it be a good idea to turn on POS HOLD (assuming GPS and a lock obviously)? At 400' on a windy day, my quad could easily end up in the river where I typically fly from. I would much rather it come directly down than drift who knows how far.
I think this would also be helpful as telemetry would show the last GPS coordinates, which would be very close to where the craft would land, instead of basically anywhere if the throttle is just reduced and it's allowed to drift.
I use RTH for failsafe, but those using LAND and have GPS I would think would rather know where their craft is instead of it just drifting to the unknown.
teckel12
on 21 Nov 2017
That would seem like a good idea and from the perspective of public safety a predictable, vertically descending craft is probably preferable to one drifting in the breeze.
I could also use RTH but wonder what happens if you don't have the battery capacity to make it back and hence I prefer to be on the ground quickly.
Similarly I would like to see the option of a low battery trigger for Failsafe SET-THR- something the craft's FC can decide all on its own and not dependent on operator, transmission link or even receiver function.
KiwiDavid
on 22 Nov 2017
@teckel12 @digitalentity It would be great if the failsafe landing would POSHOLD when possible (when GPS lock is established). How about making the failsafe landing more "intelligent"? The old behaviour - sticking to predefined throttle for predefined time - is a little bit dull, isn't it? Could it be made to utilize altitude information for an intelligent landing?
darmach
on 22 Nov 2017
@darmach Exactly. This is INAV afterall, not BF.
teckel12
on 22 Nov 2017
I would think LAND would work basically like the final part of RTH, where it would descend at a set speed with POSHOLD. Obviously, the landing altitude wouldn't be known so it couldn't change descent speed as RTH does. Once we work in the code that knows if the craft is in the air or not, it could also disable the motors on landing like RTH.
Leaving SET-THR as-is seems like good backwards compatibility as well. I see SET-THR and LAND as two different options, each which make sense and can both exist.
teckel12
on 22 Nov 2017
How's it going with this one? Maybe at least the bug with alt hold failsafe could be fixed?
darmach
on 20 Mar 2018
@darmach @KiwiDavid please re-test with 1.9
digitalentity
on 21 Mar 2018
Firmware version 1.9. Hardware SPRacing F3 clone. Test platform quad 250. Failsafe procedure is SET-THR (formerly LAND). Set Throttle value 1340
Test condition - POS HOLD and ALT HOLD then force failsafe with tx OFF.
With quad hovering at about 1.5m it behaves as expected and does fast, controlled descent to ground then does motor shut down after about 20 sec. Happy with this. Will test without Pos Hold later when wind is low and I have bigger space available.
KiwiDavid
on 22 Mar 2018
Sorry I hit the wrong button and while the issue looks like it's fixed it still needs a little more testing to confirm.
KiwiDavid
on 22 Mar 2018
Just confirming that the Set-THR/Land is also working as expected without Pos Hold engaged.
Same test as above but without Pos Hold used. Looking good.
Cheers,
David
KiwiDavid
on 22 Mar 2018
Original issue is now corrected in 1.9 and with no further input from others I think this can now be closed
KiwiDavid
on 26 Mar 2018
Related issues
jasonsmit4
路
3Comments
Jetrell
路
4Comments
ghost
路
4Comments
dankvwguy
路
4Comments
seriousrob
路
3Comments
Most helpful comment
I would think LAND would work basically like the final part of RTH, where it would descend at a set speed with POSHOLD. Obviously, the landing altitude wouldn't be known so it couldn't change descent speed as RTH does. Once we work in the code that knows if the craft is in the air or not, it could also disable the motors on landing like RTH.
Leaving SET-THR as-is seems like good backwards compatibility as well. I see SET-THR and LAND as two different options, each which make sense and can both exist.