StarlingUAS / ProjectStarling

BRL Flight Arena Infrastructure 2.0
Other
16 stars 3 forks source link

Add -f to sed in sitl.Dockerfile mavlink config modification #144

Closed mhl787156 closed 2 years ago

mhl787156 commented 2 years ago

Modified dockerfile sed to match mavlink start. Unknown how this was missed out in latest release. Also removed the starting of the payload and gimbal as they are both unnecessary and cause extra printouts.

Confusingly the following line still exists:

sitl_1             | INFO  [mavlink] MAVLink only on localhost (set param MAV_{i}_BROADCAST = 1 to enable network)

But it is unknown how to get rid of this error as, we do indeed set that parameter within the dockerfile, and it is verified that the parameter is indeed changed inside the container. But it works so moving on for now.