More enforcement on installation Gazebo 9

It exits with 255 somehow

.github/workflows/main.yml

@@ -20,7 +20,6 @@ jobs:
       - name: Build
         run: ./build.sh
       - name: Upload
-        if: ${{ github.event_name == 'release' }}
         # Not using ready S3 actions, because they either don't run on macOS machines or don't support changing the endpoint
         run: |
           AWS_ACCESS_KEY_ID=${{secrets.S3_KEY}} AWS_SECRET_ACCESS_KEY=${{secrets.S3_SECRET_KEY}} aws s3 sync . s3://clovervm/ --acl public-read --endpoint-url https://ams3.digitaloceanspaces.com --no-progress

assets/clover_vm_version

@@ -1 +0,0 @@
-unknown

assets/desktop/Clover Drone Kit Tools.desktop

@@ -1,7 +0,0 @@
-[Desktop Entry]
-Version=1.0
-Type=Link
-Name=Clover Drone Kit Tools
-Comment=Open web based Clover tools
-Icon=user-bookmarks
-URL=http://localhost/

assets/desktop/Gazebo PX4.desktop

@@ -1,8 +1,8 @@
 [Desktop Entry]
 Version=1.0
 Type=Application
-Name=Gazebo Clover
+Name=Gazebo PX4
-Comment=Run Clover drone simulation in Gazebo
+Comment=Fully-featured Gazebo simulation with default PX4 settings
 Exec=/bin/bash -c 'source /home/clover/catkin_ws/devel/setup.bash; roslaunch clover_simulation simulator.launch'
 Icon=gazebo
 Path=

assets/desktop/jMAVSim.desktop

@@ -0,0 +1,10 @@
+[Desktop Entry]
+Version=1.0
+Type=Application
+Name=jMAVSim
+Comment=A lightweight PX4 simulator
+Exec=/bin/bash -c 'cd ~/Firmware; make px4_sitl jmavsim'
+Icon=openjdk-8
+Path=
+Terminal=true
+StartupNotify=false

assets/patches/sitl_gazebo.patch

@@ -1,36 +1,41 @@
-diff --git a/src/gazebo_gps_plugin.cpp b/src/gazebo_gps_plugin.cpp
+diff --git a/include/gazebo_opticalflow_plugin.h b/include/gazebo_opticalflow_plugin.h
-index 1195f5e..6540596 100644
+index 4fbeab2..18a192b 100644
---- a/src/gazebo_gps_plugin.cpp
+--- a/include/gazebo_opticalflow_plugin.h
-+++ b/src/gazebo_gps_plugin.cpp
++++ b/include/gazebo_opticalflow_plugin.h
-@@ -41,7 +41,6 @@ GpsPlugin::~GpsPlugin()
+@@ -40,7 +40,7 @@
-   if (updateSensorConnection_)
+ #include "flow_px4.hpp"
-     updateSensorConnection_->~Connection();
-   parentSensor_.reset();
--  world_->Reset();
- }
  
- void GpsPlugin::Load(sensors::SensorPtr _parent, sdf::ElementPtr _sdf)
+ #define DEFAULT_RATE 20
-diff --git a/src/gazebo_groundtruth_plugin.cpp b/src/gazebo_groundtruth_plugin.cpp
+-#define HAS_GYRO TRUE
-index 441ebd9..dd66e12 100644
++#define HAS_GYRO true
---- a/src/gazebo_groundtruth_plugin.cpp
-+++ b/src/gazebo_groundtruth_plugin.cpp
-@@ -51,7 +51,6 @@ GroundtruthPlugin::~GroundtruthPlugin()
- {
-   if (updateConnection_)
-     updateConnection_->~Connection();
--  world_->Reset();
- }
  
- void GroundtruthPlugin::Load(physics::ModelPtr _model, sdf::ElementPtr _sdf)
+ using namespace cv;
-diff --git a/src/gazebo_lidar_plugin.cpp b/src/gazebo_lidar_plugin.cpp
+ using namespace std;
-index 38374e6..1b2fbf8 100644
+diff --git a/cmake/FindMAVLink.cmake b/cmake/FindMAVLink.cmake
---- a/src/gazebo_lidar_plugin.cpp
+index 8de3079..3fdae02 100644
-+++ b/src/gazebo_lidar_plugin.cpp
+--- a/cmake/FindMAVLink.cmake
-@@ -47,7 +47,6 @@ LidarPlugin::~LidarPlugin()
++++ b/cmake/FindMAVLink.cmake
-   newLaserScansConnection_->~Connection();
+@@ -19,6 +19,7 @@ set(_MAVLINK_EXTRA_SEARCH_HINTS
-   newLaserScansConnection_.reset();
+     ${CMAKE_SOURCE_DIR}/mavlink/
-   parentSensor_.reset();
+     ../../mavlink/
--  world_->Reset();
+     ../mavlink/
- }
++    $ENV{HOME}/Firmware/mavlink/include
+     ${CATKIN_DEVEL_PREFIX}/
+     )
  
- /////////////////////////////////////////////////
+ 
+diff --git a/package.xml b/package.xml
+index ae0fb34..12f17b8 100644
+--- a/package.xml
++++ b/package.xml
+@@ -50,8 +50,8 @@
+   <build_depend>roscpp</build_depend>
+   <build_depend>sensor_msgs</build_depend>
+   <build_depend>std_msgs</build_depend>
+-  <build_depend>python-rospkg</build_depend>
+-  <build_depend>python-jinja2</build_depend>
++  <build_depend>python3-rospkg</build_depend>
++  <build_depend>python3-jinja2</build_depend>
+   <run_depend>eigen</run_depend>
+   <run_depend>gazebo_ros</run_depend>
+   <run_depend>geometry_msgs</run_depend>

build.sh

@@ -5,18 +5,6 @@ set -ex
 # FIXME: Use system Packer if possible
 #PACKER=$(which packer)
 
-echo "--- Fill version asset"
-# if [[ $GITHUB_REF == refs/tags/*-rc* ]]; then
-    # remove rc label
-    # VERSION=${GITHUB_REF#refs/tags/}
-    # VERSION=${VERSION/-rc*/}
-if [[ $GITHUB_REF == refs/tags/* ]]; then
-    VERSION=${GITHUB_REF#refs/tags/}
-else
-    VERSION=$(git describe --always)
-fi
-echo $VERSION > assets/clover_vm_version
-
 #if [ "x${PACKER}" == "x" ]; then
 PACKER="./packer"
 if [ ! -f ${PACKER} ]; then
@@ -44,6 +32,16 @@ ${PACKER} build ros_ide_vm.json
 
 echo "--- Marking the VM"
 
+# if [[ $GITHUB_REF == refs/tags/*-rc* ]]; then
+    # remove rc label
+    # VERSION=${GITHUB_REF#refs/tags/}
+    # VERSION=${VERSION/-rc*/}
+if [[ $GITHUB_REF == refs/tags/* ]]; then
+    VERSION=${GITHUB_REF#refs/tags/}
+else
+    VERSION=$(git describe --always)
+fi
+
 VM_NAME="clover-devel_${VERSION}.ova"
 mv ./output-virtualbox-ovf/clover-devel.ova ./output-virtualbox-ovf/${VM_NAME}
 ls -l output-virtualbox-ovf

ros_ide_vm.json

@@ -63,11 +63,6 @@
     "source": "{{user `assetsDir`}}/patches",
     "destination": "/tmp"
   },
-  {
-    "type": "file",
-    "source": "{{user `assetsDir`}}/clover_vm_version",
-    "destination": "/tmp/clover_vm_version"
-  },
   {
     "type": "shell",
     "script": "scripts/install_software.sh",

scripts/install_software.sh

@@ -12,89 +12,50 @@ echo "--- Increasing apt retries"
 sudo -E sh -c 'echo "APT::Acquire::Retries \"3\";" > /etc/apt/apt.conf.d/80-retries'
 cat /etc/apt/apt.conf.d/80-retries
 
-echo "--- Updating apt"
-sudo -E sh -c 'apt-get update'
-
 echo "--- Allowing apt to perform its updates"
-sudo -E sh -c 'while fuser /var/lib/dpkg/lock ; do sleep 0.5 ; done'
+sudo -E sh -c 'apt-get update; while fuser /var/lib/dpkg/lock ; do sleep 0.5 ; done'
 
 echo "--- Installing open-vm-tools"
-sudo -E sh -c 'apt-get install -y open-vm-tools open-vm-tools-desktop'
 
-echo "--- Installing ROS desktop"
+sudo -E sh -c 'apt-get update; apt-get install -y open-vm-tools open-vm-tools-desktop'
-sudo -E sh -c 'apt-get install -y curl'
-sudo sh -c 'echo "deb http://packages.ros.org/ros/ubuntu $(lsb_release -sc) main" > /etc/apt/sources.list.d/ros-latest.list'
-curl -s https://raw.githubusercontent.com/ros/rosdistro/master/ros.asc | sudo apt-key add -
-sudo -E sh -c 'apt-get update; apt-get install -y python3-pip python3-rosdep python3-rosinstall-generator python3-wstool build-essential ros-noetic-desktop'
 
-echo "source /opt/ros/noetic/setup.bash" >> ~/.bashrc
+echo "--- Installing ROS desktop packages"
-source /opt/ros/noetic/setup.bash
+
+sudo -E sh -c 'echo "deb http://packages.ros.org/ros/ubuntu $(lsb_release -sc) main" > /etc/apt/sources.list.d/ros-latest.list'
+sudo -E sh -c 'apt-key adv --keyserver "hkp://keyserver.ubuntu.com:80" --recv-key C1CF6E31E6BADE8868B172B4F42ED6FBAB17C654'
+sudo -E sh -c 'apt-get update; apt-get install -y python3-rosdep python3-rosinstall-generator python3-wstool build-essential ros-noetic-desktop'
 
-echo "--- Updating rosdep"
 sudo -E sh -c 'rosdep init'
 rosdep update
 
-echo "--- Creating Catkin workspace"
+# FIXME: PX4 needs pip to be installed
-mkdir -p ~/catkin_ws/src
+# FIXME: python2 dependencies?
-cd ~/catkin_ws
+echo "--- Downloading PX4 and installing its dependencies"
-catkin_make
-echo "source ~/catkin_ws/devel/setup.bash" >> ~/.bashrc
-source ~/catkin_ws/devel/setup.bash
 
-echo "--- Getting Clover sources"
+# v1.11.1, TODO: update to v1.12, remove Gazebo downgrade
-cd ~/catkin_ws/src
+#git clone --recursive -b v1.11.1-clover https://github.com/CopterExpress/Firmware ${HOME}/Firmware
-git clone --depth 1 https://github.com/CopterExpress/clover
+## PX4 v1.11.1 script will happily run sudo by itself
-git clone --depth 1 https://github.com/CopterExpress/ros_led
+#${HOME}/Firmware/Tools/setup/ubuntu.sh
-git clone --depth 1 https://github.com/ethz-asl/mav_comm
+## Ubuntu 20.04 no longer sets assistive_technologies, thankfully
 
-echo "--- Installing dependencies with rosdep"
+# v1.10.1:
-cd ~/catkin_ws
+git clone -b v1.10.1-clover https://github.com/CopterExpress/Firmware ${HOME}/Firmware
-rosdep install --from-paths src --ignore-src -y
+sudo -E sh -c 'apt-get update; apt-get install -y python3-pip'
+sudo -E -S sh -c '${HOME}/Firmware/Tools/setup/ubuntu.sh'
+sudo -E -S sh -c 'echo "2" | update-alternatives --config java'
+sudo -E -S sed -i -e '/^assistive_technologies=/s/^/#/' /etc/java-*-openjdk/accessibility.properties
 
-echo "--- Installing Clover's Python dependencies"
+echo "--- Prebuilding PX4 SITL configuration"
-sudo -E sh -c '/usr/bin/python3 -m pip install -r ~/catkin_ws/src/clover/clover/requirements.txt'
+make -C ${HOME}/Firmware px4_sitl
-
+echo "--- Patching gazebo plugins for SITL"
-echo "--- Downloading PX4"
+cd ${HOME}/Firmware/Tools/sitl_gazebo
-git clone --recursive --depth 1 --branch v1.13.0 https://github.com/PX4/PX4-Autopilot.git ~/PX4-Autopilot
-ln -s ~/PX4-Autopilot ~/catkin_ws/src/
-ln -s ~/PX4-Autopilot/Tools/sitl_gazebo ~/catkin_ws/src/
-#ln -s ~/PX4-Autopilot/src/modules/mavlink/mavlink ~/catkin_ws/src/
-#git clone --depth 1 https://github.com/mavlink/c_library_v2.git ~/catkin_ws/src/mavlink/ # FIXME:
-
-echo "--- Installing PX4 dependencies"
-~/PX4-Autopilot/Tools/setup/ubuntu.sh
-pip3 install --user toml
-sudo -E sh -c 'apt-get install -y ant openjdk-11-jdk' # Additional packages for jMAVSim
-
-echo "--- Patching mavlink_sitl_gazebo"
-# See https://github.com/PX4/PX4-SITL_gazebo/pull/872
-cd ~/PX4-Autopilot/Tools/sitl_gazebo
 patch -p1 < /tmp/patches/sitl_gazebo.patch
+echo 'export SVGA_VGPU10=0' >> ${HOME}/Firmware/Tools/setup_gazebo.bash
 
-echo "--- Addding Gazebo initialization to bashrc"
+echo "source /opt/ros/noetic/setup.bash" >> ~/.bashrc
-echo "source /usr/share/gazebo/setup.sh" >> ~/.bashrc
-echo "export SVGA_VGPU10=0" >> ~/.bashrc
-
-echo "--- Addding Clover airframe"
-ln -s ~/catkin_ws/src/clover/clover_simulation/airframes/* ~/PX4-Autopilot/ROMFS/px4fmu_common/init.d-posix/airframes/
-
-echo "--- Installing geographiclib datasets"
-sudo -E sh -c '/opt/ros/noetic/lib/mavros/install_geographiclib_datasets.sh'
-
-echo "--- Build PX4"
-cd ~/PX4-Autopilot
-make px4_sitl
-
-echo "--- Building the workspace"
-# sudo -E sh -c 'rm -rf /opt/ros/noetic/include/mavlink'
-ls ~/catkin_ws/src/PX4-Autopilot/build/px4_sitl_default/mavlink
-echo 'include_directories("/home/clover/PX4-Autopilot/build/px4_sitl_default/mavlink")' >> ~/catkin_ws/src/sitl_gazebo/CMakeLists.txt
-sed -i 's|#include <mavlink/mavlink_types.h>|#include <mavlink_types.h>|' ~/catkin_ws/src/PX4-Autopilot/src/modules/mavlink/mavlink_bridge_header.h
-cd ~/catkin_ws
-catkin_make mavlink_c_generate
-catkin_make
 
 echo "--- Installing Visual Studio Code"
+
 sudo -E sh -c 'apt-get update; apt-get install -y curl'
 curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > ${HOME}/packages.microsoft.gpg
 sudo -E sh -c 'install -o root -g root -m 644 ${HOME}/packages.microsoft.gpg /usr/share/keyrings'
@@ -107,15 +68,53 @@ code --install-extension ms-vscode.cmake-tools
 code --install-extension ms-vscode.cpptools
 code --install-extension streetsidesoftware.code-spell-checker
 code --install-extension eamodio.gitlens
+echo "--- Installing pylint"
+/usr/bin/python3 -m pip install -U pylint --user
 
-echo "--- Installing additional Python packages"
+echo "--- Cloning and installing Clover packages"
-/usr/bin/python3 -m pip install -U pylint pyzbar --user
+sudo sh -c 'curl http://deb.coex.tech/aptly_repo_signing.key 2> /dev/null | apt-key add -'
+sudo sh -c 'echo "deb http://deb.coex.tech/ros xenial main" > /etc/apt/sources.list.d/coex.tech.list'
+sudo sh -c 'echo "yaml file:///etc/ros/rosdep/coex.yaml" > /etc/ros/rosdep/sources.list.d/99-coex.list'
+sudo sh -c 'cat <<EOF > /etc/ros/rosdep/coex.yaml
+led_msgs:
+  ubuntu:
+    focal: [ros-noetic-led-msgs]
+async_web_server_cpp:
+  ubuntu:
+    focal: [ros-noetic-async-web-server-cpp]
+ros_pytest:
+  ubuntu:
+    focal: [ros-noetic-ros-pytest]
+tf2_web_republisher:
+  ubuntu:
+    focal: [ros-noetic-tf2-web-republisher]
+web_video_server:
+  ubuntu:
+    focal: [ros-noetic-web-video-server]
+ws281x:
+  ubuntu:
+    focal: [ros-noetic-ws281x]
+EOF'
+sudo apt-get update
+rosdep update
+mkdir -p ${HOME}/catkin_ws/src
+git clone https://github.com/CopterExpress/clover ${HOME}/catkin_ws/src/clover
+git clone https://github.com/CopterExpress/ros_led ${HOME}/catkin_ws/src/ros_led
+# These packages are missing from Noetic release, but are required for sitl_gazebo
+git clone https://github.com/ethz-asl/mav_comm ${HOME}/catkin_ws/src/mav_comm
+# Make PX4 and Gazebo plugins visible in the workspace
+ln -s ${HOME}/Firmware ${HOME}/catkin_ws/src/Firmware
+ln -s ${HOME}/Firmware/Tools/sitl_gazebo ${HOME}/catkin_ws/src/sitl_gazebo
+rosdep install --from-paths ${HOME}/catkin_ws/src --ignore-src --rosdistro noetic -y
+sudo /opt/ros/noetic/lib/mavros/install_geographiclib_datasets.sh
+sudo /usr/bin/python3 -m pip install -r ${HOME}/catkin_ws/src/clover/clover/requirements.txt
+source /opt/ros/noetic/setup.bash
+cd ${HOME}/catkin_ws && catkin_make
+echo "source ${HOME}/catkin_ws/devel/setup.bash" >> ~/.bashrc
 
-echo "--- Exposing examples"
+sudo -E sh -c 'apt-get update; apt-get install -y gazebo9' # downgrade Gazebo to 9 to make PX4 v1.10.1 work
-ln -s ${HOME}/catkin_ws/src/clover/clover/examples ${HOME}/
-[[ -d ${HOME}/examples ]] # test symlink is valid
 
-echo "--- Installing npm and building documentation"
+echo "--- Installing npm"
 cd ${HOME}
 NODE_VERSION=v10.15.0 # GitBook won't install on newer version
 wget --progress=dot:giga https://nodejs.org/dist/$NODE_VERSION/node-$NODE_VERSION-linux-x64.tar.gz
@@ -134,6 +133,10 @@ gitbook install
 gitbook build
 touch node_modules/CATKIN_IGNORE docs/CATKIN_IGNORE _book/CATKIN_IGNORE clover/www/CATKIN_IGNORE # ignore documentation files by catkin
 
+echo "--- Exposing examples"
+ln -s ${HOME}/catkin_ws/src/clover/clover/examples ${HOME}/
+[[ -d ${HOME}/examples ]] # test symlink is valid
+
 echo "--- Enabling roscore service"
 sed -i "s/pi/${USER}/g" ${HOME}/catkin_ws/src/clover/builder/assets/roscore.service
 sudo cp ${HOME}/catkin_ws/src/clover/builder/assets/roscore.service /etc/systemd/system
@@ -151,7 +154,6 @@ sudo -E sh -c 'apt-get update; apt-get install -y firefox'
 echo "--- Installing Monkey web server"
 sudo apt-get install -y /tmp/packages/monkey_1.6.9-1_amd64.deb
 sed "s/pi/${USER}/g" ${HOME}/catkin_ws/src/clover/builder/assets/monkey | sudo tee /etc/monkey/sites/default
-sudo -E sh -c "sed -i 's/SymLink Off/SymLink On/' /etc/monkey/monkey.conf"
 sudo cp ${HOME}/catkin_ws/src/clover/builder/assets/monkey.service /etc/systemd/system/monkey.service
 sudo systemctl enable monkey
 
@@ -164,10 +166,6 @@ sudo -E sh -c 'hostnamectl set-hostname clover-dev; sed -i "s/ubuntu/clover-dev
 echo "export ROS_HOSTNAME=\`hostname\`.local" >> ${HOME}/.bashrc
 chmod a+x ${HOME}/Desktop/*
 
-echo "--- Creating /etc/clover_vm_version"
-sudo -E sh -c 'mv /tmp/clover_vm_version /etc/clover_vm_version'
-cat /etc/clover_vm_version
-
 echo "--- Cleaning up"
 sudo -E sh -c 'apt-get -y autoremove; apt-get -y autoclean; apt-get -y clean; fstrim -v /'
 
@@ -177,7 +175,7 @@ python2 --version
 python3 --version
 # ipython --version
 ipython3 --version
-gazebo --version || true # FIXME: Gazebo exits with 255 on --version somehow
+gazebo --version || true # FIXME: Gazebo exist with 255 on --version somehow
 node -v
 npm -v
 byobu --version
@@ -187,10 +185,9 @@ pip --version
 pip3 --version
 monkey --version
 systemctl --version
-# TODO: add Python tests
 
+source $HOME/catkin_ws/devel/setup.bash
 roscore -h
-rosversion px4
 rosversion clover
 rosversion aruco_pose
 rosversion mavros
@@ -205,18 +202,5 @@ rosversion cv_camera
 rosversion web_video_server
 rosversion nodelet
 
-echo "--- Run Clover's Python libraries validation"
-$HOME/catkin_ws/src/clover/builder/test/tests_py3.py
-
-echo "--- Versions of all installed ROS packages"
-set +x
-rospack list-names | while read line; do echo $line `rosversion $line`; done
-set -x
-
 echo "Trying running the Gazebo simulator, check the output"
 timeout --preserve-status 30 roslaunch clover_simulation simulator.launch gui:=false --screen
-
-echo "Trying running jMAVSim, check the output"
-# cd ~/PX4-Autopilot
-# HEADLESS=1 timeout --preserve-status 30 make px4_sitl jmavsim
-HEADLESS=1 timeout --preserve-status 30 roslaunch clover_simulation simulator.launch type:=jmavsim gui:=false --screen