ArduPilot / ArduRemoteID

RemoteID support using OpenDroneID
GNU General Public License v2.0
102 stars 50 forks source link

allow lock_level to set to -1 #111

Closed BluemarkInnovations closed 1 year ago

BluemarkInnovations commented 1 year ago

In PR91, the lock_level -1 was introduced. However, the parameter did not allow to set it to negative values, because it was defined as uint8_t.

This PR converts the lock_level to int8_t.

Via DroneCAN GUI tool I can set it to -1. Via Mission Planner, it doesn't work. Not sure if something is missing in this PR or if it is a limitation of Mission Planner/MAVLink.

tridge commented 1 year ago

looks good, thanks! just needs some indication of testing working

BluemarkInnovations commented 1 year ago

FYI.

Screenshot_2023-06-16_10-37-00 Screenshot_2023-06-16_10-37-09