forked from IntelRealSense/realsense-ros
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
047d601
commit fde7c26
Showing
2 changed files
with
20 additions
and
9 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -25,7 +25,7 @@ jobs: | |
strategy: | ||
fail-fast: false | ||
matrix: | ||
ros_distro: [rolling, iron, humble, foxy] | ||
ros_distro: [rolling, iron, humble, foxy, jazzy] | ||
include: | ||
- ros_distro: 'rolling' | ||
os: ubuntu-22.04 | ||
|
@@ -35,6 +35,8 @@ jobs: | |
os: ubuntu-22.04 | ||
- ros_distro: 'foxy' | ||
os: ubuntu-20.04 | ||
- ros_distro: 'jazzy' | ||
os: ubuntu-24.04 | ||
|
||
steps: | ||
|
||
|
@@ -53,14 +55,14 @@ jobs: | |
./pr_check.sh | ||
# [email protected] is the last version supporting foxy (EOL) | ||
# [email protected] is needed to support humble/iron/rolling | ||
# [email protected] is needed to support humble/iron/rolling/jazzy | ||
# so, seperating steps with if conditions | ||
- name: build ROS2 for foxy | ||
if: ${{ matrix.ros_distro == 'foxy' }} | ||
uses: ros-tooling/[email protected] | ||
with: | ||
required-ros-distributions: ${{ matrix.ros_distro }} | ||
- name: build ROS2 for humble/iron/rolling | ||
- name: build ROS2 for humble/iron/rolling/jazzy | ||
if: ${{ matrix.ros_distro != 'foxy' }} | ||
uses: ros-tooling/[email protected] | ||
with: | ||
|
@@ -92,13 +94,13 @@ jobs: | |
echo "================= ROSDEP UPDATE =====================" | ||
# temp fix for rolling sources.. TODO: track when we can remove the two commands below | ||
# see https://discourse.ros.org/t/psa-rolling-ci-or-docker-build-fix-from-rosdep-errors-in-24-04-transition/36902 | ||
sudo sed -i "s|ros\/rosdistro\/master|ros\/rosdistro\/rolling\/2024-02-28|" /etc/ros/rosdep/sources.list.d/20-default.list | ||
export ROSDISTRO_INDEX_URL=https://raw.githubusercontent.com/ros/rosdistro/rolling/2024-02-28/index-v4.yaml | ||
sudo sed -i "s|ros\/rosdistro\/master|ros\/rosdistro\/rolling\/2024-04-30|" /etc/ros/rosdep/sources.list.d/20-default.list | ||
export ROSDISTRO_INDEX_URL=https://raw.githubusercontent.com/ros/rosdistro/rolling/2024-04-30/index-v4.yaml | ||
rosdep update --rosdistro ${{ matrix.ros_distro }} --include-eol-distros | ||
echo "================= ROSDEP INSTALL ====================" | ||
rosdep install -i --reinstall --from-path src --rosdistro ${{ matrix.ros_distro }} --skip-keys=librealsense2 -y | ||
echo "================== COLCON BUILD ======================" | ||
colcon build --cmake-args '-DBUILD_TOOLS=ON' | ||
colcon build --cmake-args '-DBUILD_TOOLS=ON' --no-warn-unused-cli | ||
## This step is commented out since we don't use rosbag files in "Run Tests" step below. | ||
## Please uncomment when "Run Tests" step is fixed to run all tests. | ||
|
@@ -114,9 +116,13 @@ jobs: | |
|
||
- name: Install Packages For Tests | ||
run: | | ||
sudo apt install python3-venv | ||
python3 -m venv .venv | ||
source .venv/bin/activate | ||
sudo apt-get install python3-pip | ||
pip3 install numpy --upgrade | ||
pip3 install numpy-quaternion tqdm | ||
pip3 install numpy-quaternion tqdm pyyaml | ||
- name: Run Tests | ||
run: | | ||
|
@@ -126,7 +132,8 @@ jobs: | |
# the next command might be needed for foxy distro, since this package is not installed | ||
# by default in ubuntu 20.04. For other distro, the apt install command will be ignored. | ||
sudo apt install -y ros-${{matrix.ros_distro}}-sensor-msgs-py | ||
python3 src/realsense-ros/realsense2_camera/scripts/rs2_test.py non_existent_file | ||
source ../.venv/bin/activate | ||
../.venv/bin/python3 src/realsense-ros/realsense2_camera/scripts/rs2_test.py non_existent_file | ||
# don't run integration tests for foxy since some testing dependecies packages like | ||
# tf_ros_py are not avaialble | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters