intel / collision-avoidance-library

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

Update testbed #1

Closed guiccbr closed 7 years ago

dakerfp commented 7 years ago

LGTM

dakerfp commented 7 years ago

Why do you need a second connection?

guiccbr commented 7 years ago

@dakerfp, both gzitl and mavquadcopter must communicate in parallel with arducopter. This second connection is for the mavlink_vehicles instance present in mavquadcopter. I know that it might look misplaced, since we're not running the collision avoidance gcs from the script, but I'll do that before merging. I'm still having problems to configure sitl to send packages to two channels without mavproxy, that's why I haven't added that call yet.

edit: b1ad616 fixes the parameters on the eeprom to allow arducopter to send messages to two gcss at the same time. 8f9bb91 adds the call to coav_gcs (who uses that second connection) to the script.

rchiossi commented 7 years ago

LGTM