-
Notifications
You must be signed in to change notification settings - Fork 13.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
Use UTM_GLOBAL_POSITION for collision avoidance #12452
Use UTM_GLOBAL_POSITION for collision avoidance #12452
Conversation
154a664
to
f054426
Compare
f054426
to
91d6513
Compare
91d6513
to
803c043
Compare
803c043
to
6c04e9a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The basics here look fine (safe to merge), but I'm not up to speed with the overall design wrt ADSB.
@dagar Essentially ADBS/FLARM have an incoming message with validity flags, position and heading. This is published as a UTM_GLOBAL_POSITION is a bit like transponder message, but it also includes next waypoint info. All this PR does is fake a transponder message: drop the next waypoint, fake the validity flags, and convert the current position into transponder units/format. If we aren't ever going to use the next waypoint info and we don't want to be able to differentiate the source of the messages this is all that is needed. Personally I think it is worth changing the topic from Why? Because I think we might want to provide more stringent handling for ADBS than we do for UTM. |
@dk7xe are you able to review this? |
you mean the code? no.. i don't think i'm capable.. i can understand C but being far from calling myself knowledgeable. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. @ItsTimmy what's the testing level here?
@bkueng I sent some |
Based on Issue #11262.
Describe problem solved by the proposed feature
At the moment there is no collision avoidance feature based on position reports from other drones available. The only collision avoidance is done for manned aircraft traffic based on ADSB_VEHICLE message provided by a ADSB receiver connected to FMU.
With considering other drone positions as well for collision avoidance drone operation safety can be significantly improved when multiple drones share the same airspace.
UTM_GLOBAL_POSITION message broadcasted by a drone and implemented with #10873 provides all the necessary information similar to ADSB_VEHICLE.
Describe your preferred solution
Use UTM_GLOBAL_POSITION message similar to ADSB_VEHICLE for collision avoidance. A handler was added to
mavlink_receiver.cpp
which translates allUTM_GLOBAL_POSITON
messages into UOrbtransponder_report
messages.Additional context
Cooperative awareness is mandatory in future UTM concepts to allow multiple drone operations within same airspace.