-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Remote ID docs to PX4 v1.14 level #2605
Comments
I have went through relevant resources and checked implementations in both sides (ArduPilot & PX4 with Andy's PR), and referencing your Issue to create a separate Issue to keep track of Remote ID development effort (to keep it separate from docs focused one here)! For your questions, I asked on discord thread , and I will bring up how much support we will have for Remote ID in v1.14 in maintainers channel 🙏 I should have kept myself on top of this, but I'm now getting myself familiar with it, thank you for this push! |
Thanks for the help! |
Hi, here are my comments.
-Who is responsible for storing and streaming the basic ID? In the ArduPilot world, the basic ID is stored or as parameter in an ArduRemoteID device or it will be streamed by GCS software such as QGroundControl. QGroundControl needs to check a valid SN number. Also ArduRemoteID (RemoteID transponder) needs to do that. See ArduPilot/ArduRemoteID#101. This PR has to be merged.
Remote ID needs to be tamper resistance. In ArduPilot, they have chosen for a locked bootloader of the autopilot system and you can also lock the ArduRemoteID module to prevent unauthorized changes. So my point is, will this be enough tamper resistance? In my opinion, for most use cases it is. But if it is a parameter than can easy be modified by the end-user this is not sufficient in my opinion.
More ore less. If it is tamper resistance and you stream the system and location messages every second. And read OPEN_DRONE_ID_ARM_STATUS to determine if you are allowed to arm. Then I would say you are 'compliant ready'. In F3586 each UAV vendor needs to do several tests that show that you are compliant to Remote ID. Once of them is that the antenna pattern needs to be (sufficient) omni-directional. Another is that the GNSS hardware provides the correct location data using some basic tests. These tests are outside the PX4 or ArduPilot implementation.
OPEN_DRONE_ID_ARM_STATUS is mandatory for US compliance. OPEN_DRONE_ID_OPERATOR_ID is only needed for the EU. OPEN_DRONE_ID_SELF_ID is optional. Also I need to check PX4/PX4-Autopilot#21647, but according to FAA regulation, operator location needs to be fixed or dynamic. Value take-off position is only valid for Remote ID broadcast modules. |
Note, I just had discussion with Ramon today and concluded that we don't have a good understanding/agreement on the implementation of Remote ID, and will need to organize a working group with manufacturers & people from the companies to discuss the details. Hence, Remote ID would be referenced as an 'experimental feature' for v1.14, and we will have it more complete for the next release. We will discuss this in maintainers meeting tomorrow too, and I will provide further updates 🙏 |
Thanks @BluemarkInnovations - really helpful.
@junwoo091400 This is a sensible position to take. Thanks for kicking off the broader discussion.
|
Since noone in the maintainers were confident about current Remote ID implementation, this won't be going into v1.14. |
I pushed for having a specific deadline for the release on the last maintainer's call, but the maintainers weren't sure about having that, since we still have PR/Issues to resolve. We did discuss however that from next release, we should have the timeline and more heads-up on when to stop taking in features, so that we don't get pending PR/Issues remaining for months like this time 😞 |
This issue has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: https://discuss.px4.io/t/remote-id-coordination-july-11th/33191/1 |
RemoteID/OpenID mavlink protocol has been added to
main
.In mainline the current implementation :
Streams these streamed at 1Hz in modes: normal, onboard, usb, onboard low bandwidth:
This has a stream defined but is not streamed by default:
There is one parameter/setting:
RemoteID hardware presence determined by existence of heartbeat message in last 3 seconds.
Remote ID hardware
health
is determined by heartbeat too (spec indicates should also depend onOPEN_DRONE_ID_ARM_STATUS
, which is not handled on receive).No incoming drone ID messages are handled.
There is also a new PR Open Drone ID Live GNSS and Arm status prototype #21647.
Other docs resources:
That about right?
The text was updated successfully, but these errors were encountered: