search

ArduPilot / ArduRemoteID

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

OpenDroneID - Need to add ENUM types for MAV_ODID_OPERATOR_ID_TYPE #33

Closed sborenstein-ift closed 2 years ago

sborenstein-ift commented 2 years ago

This will be a MAVLink issue.

Currently, MAV_ODID_ID_TYPE only contains CAA. Need to add other options.

friissoren commented 2 years ago

Sorry, could you elaborate?

MAVLink has the available enum values

And those are there in the ArduPilot variant also.

In the core-c library, the corresponding ones are here

sborenstein-ift commented 2 years ago

Whoops... this is supposed to be fore MAV_ODID_OPERATOR_ID_TYPE.

friissoren commented 2 years ago

Sorry, I still don't understand what is missing.

MAVLink has it defined here. And core-c here. ArduRemoteID using it here.

The standards do not have any other values defined than zero for this enum field. Unless you want to spell out the "Reserved" and "Available for private use" ranges.

tridge commented 2 years ago

this seems to be fine