ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

need to vet parameter set() calls

Open tridge opened this issue 3 years ago • 2 comments

we have quite a lot of code using .set() on parameters to temporarily disable features. Examples in mode_turtle and motor_test in copter with failsafes. if the user fetches parameters while these operations are happening they will end up with a param set without RC failsafe enabled, which if loaded later could be dangerous

tridge avatar Aug 03 '22 08:08 tridge

https://github.com/ArduPilot/ardupilot/issues/20788

IamPete1 avatar Aug 05 '22 22:08 IamPete1

As discussed on the call perhaps we could create an new AP_Param type that creates a shadow variable that is used from within the C++ code when .get() or .set() are called but not used for the param download.

rmackay9 avatar Aug 09 '22 00:08 rmackay9