intel / collision-avoidance-library

A framework for testing and benchmarking collision avoidance strategies
Apache License 2.0
81 stars 39 forks source link

Add PX4 support #26

Closed guiccbr closed 7 years ago

guiccbr commented 7 years ago

Testbed now connects to PX4 instead of Ardupilot when PX4_FLIGHT_STACK is set. Also, some magic numbers have been replaced by variables.

Reviewed version of #23

guiccbr commented 7 years ago

@lfelipe suggested to change USE_PX4_AUTOPILOT to something more generic and to keep PX4 as default on testbed.

I'll make those changes and fix conflicts before merging. @rchiossi, do you have anything else to add?

rchiossi commented 7 years ago

LGTM. Squash and merge it as is and opena new pr to change the default to PX4, since that will also require you to update the readme. Add the change in the USE_PX4_AUTOPILOT to this new PR.

guiccbr commented 7 years ago

Merging...