Skip to content

Commit 612edc9

Browse files
committed
add hydra_ros ci workflow and fix qos issue
1 parent aace053 commit 612edc9

File tree

4 files changed

+44
-50
lines changed

4 files changed

+44
-50
lines changed

.github/workflows/ci.yaml

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
---
2+
# based on https://github.com/actions/starter-workflows/blob/main/ci/cmake-single-platform.yml
3+
name: Hydra Build and Test
4+
on: {push: {branches: [main]}, pull_request: {}}
5+
jobs:
6+
lint:
7+
runs-on: ubuntu-latest
8+
steps:
9+
- uses: actions/checkout@v4
10+
- name: Dependencies
11+
run: sudo apt-get update && sudo apt install pipx
12+
- name: Lint
13+
run: pipx install pre-commit && cd ${{github.workspace}} && pre-commit run --all-files
14+
ros2:
15+
runs-on: ubuntu-latest
16+
container: ros:jazzy-perception
17+
steps:
18+
- uses: actions/checkout@v4
19+
with: {path: src/hydra_ros}
20+
- name: CCache
21+
id: cache-ccache
22+
uses: actions/cache@v4
23+
with:
24+
path: ~/.cache/ccache
25+
key: ${{ runner.os }}-ccache-${{ github.sha}}
26+
restore-keys: |
27+
${{ runner.os }}-ccache-${{ github.sha}}
28+
${{ runner.os }}-ccache-
29+
- name: Dependencies
30+
run: |
31+
apt update && apt install -y python3-vcstool git ccache
32+
vcs import src < src/hydra_ros/install/packages.yaml
33+
rosdep update --rosdistro jazzy && rosdep install --rosdistro jazzy --from-paths src --ignore-src -r -y
34+
- name: Build
35+
shell: bash
36+
run: |
37+
source /opt/ros/jazzy/setup.bash
38+
colcon build --cmake-args --no-warn-unused-cli -DCMAKE_BUILD_TYPE=Release -DSEMANTIC_INFERENCE_USE_TRT=OFF -DCMAKE_CXX_COMPILER_LAUNCHER=ccache -DCMAKE_C_COMPILER_LAUNCHER=ccache
39+
- name: Test
40+
shell: bash
41+
run: |-
42+
source /opt/ros/jazzy/setup.bash
43+
colcon test --packages-select hydra_ros hydra_visualizer --return-code-on-test-failure --ctest-args --rerun-failed --output-on-failure --event-handlers console_cohesion+

hydra_ros/src/input/ros_sensors.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ std::optional<sensor_msgs::msg::CameraInfo> getCameraInfo(const RosCamera::Confi
7878
<< " to initialize sensor model";
7979

8080
const auto start = nh.now();
81-
const auto qos = rclcpp::QoS(1);
81+
const auto qos = rclcpp::QoS(1).durability(rclcpp::DurabilityPolicy::BestAvailable);
8282
const size_t timeout = std::floor(c.warning_timeout_s * 1000);
8383

8484
std::optional<sensor_msgs::msg::CameraInfo> msg;
Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,6 @@ repositories:
33
type: git
44
url: https://github.com/MIT-SPARK/config_utilities.git
55
version: main
6-
hydra:
7-
type: git
8-
url: https://github.com/MIT-SPARK/Hydra.git
9-
version: main
106
hydra_ros:
117
type: git
128
url: https://github.com/MIT-SPARK/Hydra-ROS.git

install/ros2.yaml

Lines changed: 0 additions & 45 deletions
This file was deleted.

0 commit comments

Comments
 (0)