Skip to content
Merged
Show file tree
Hide file tree
Changes from 14 commits
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
16c37cb
Fix devcontainer
tonynajjar May 4, 2024
6ce2187
fix
tonynajjar May 4, 2024
fc3c5fe
.
tonynajjar May 4, 2024
5045bd7
.
tonynajjar May 4, 2024
40976ea
.
tonynajjar May 4, 2024
a582494
fix
tonynajjar May 6, 2024
56372f0
pr comments
tonynajjar May 6, 2024
481b777
bust cache
tonynajjar May 6, 2024
17b434e
test with docker image
tonynajjar May 6, 2024
6699cb5
Revert "test with docker image"
tonynajjar May 6, 2024
476050c
Merge branch 'ros-navigation:main' into fix-devcontainer
tonynajjar May 15, 2024
ffdd515
fixes
tonynajjar May 15, 2024
e451b1c
fixes
tonynajjar May 15, 2024
f17374e
remove comment
tonynajjar May 15, 2024
abda167
Update URI for new GitHub org and GHCR repo
ruffsl May 17, 2024
1c5a90c
replace fix
tonynajjar May 18, 2024
10bade2
replace fix
tonynajjar May 18, 2024
fd719ff
ignore maybe uninitialized error
tonynajjar May 18, 2024
75a6282
inline comment
tonynajjar May 21, 2024
b676fc5
fix linting
tonynajjar May 21, 2024
7c0a9ac
Add comments
tonynajjar May 21, 2024
6fc3344
fix bt test
tonynajjar May 21, 2024
7034a1d
fix nav2 test
tonynajjar May 21, 2024
32dfd1c
fix waypoint follower test
tonynajjar May 21, 2024
ebd9fcb
More improvements towards Jazzy migration
SteveMacenski May 21, 2024
2b48c37
fix more tests
SteveMacenski May 22, 2024
d7b3c83
do a few more
SteveMacenski May 22, 2024
974e051
last for the day
SteveMacenski May 22, 2024
f77277f
fix remaining BT tests
SteveMacenski May 22, 2024
19940fb
fixing last test issues
SteveMacenski May 22, 2024
7d9aa4c
Merge remote-tracking branch 'upstream/BR2' into fix-devcontainer
tonynajjar May 23, 2024
4bc062c
change linting image
tonynajjar May 23, 2024
f671d38
comment out tests needing gazebo
tonynajjar May 23, 2024
1b559a8
Merge remote-tracking branch 'origin/main' into fix-devcontainer
tonynajjar May 23, 2024
c3ecef9
break the cache cuz why not
tonynajjar May 23, 2024
93bd849
fix dynamic param test
tonynajjar May 23, 2024
ffb9ea0
revert changes to test_bt_utils
tonynajjar May 23, 2024
12588f5
fix dyn param
tonynajjar May 23, 2024
80d5238
comment out turtlebot3_gazebo
tonynajjar May 28, 2024
fdd0660
test with spin_all
tonynajjar May 28, 2024
ffb8584
Update Dockerfile
SteveMacenski May 28, 2024
8246c78
Update .circleci/config.yml
SteveMacenski May 28, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 5 additions & 8 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,12 @@ _commands:
- restore_cache:
name: Restore Cache << parameters.key >>
keys:
- "<< parameters.key >>-v21\
- "<< parameters.key >>-v22\
-{{ arch }}\
-{{ .Branch }}\
-{{ .Environment.CIRCLE_PR_NUMBER }}\
-{{ checksum \"<< parameters.workspace >>/lockfile.txt\" }}"
- "<< parameters.key >>-v21\
- "<< parameters.key >>-v22\
-{{ arch }}\
-main\
-<no value>\
Expand All @@ -58,7 +58,7 @@ _commands:
steps:
- save_cache:
name: Save Cache << parameters.key >>
key: "<< parameters.key >>-v21\
key: "<< parameters.key >>-v22\
-{{ arch }}\
-{{ .Branch }}\
-{{ .Environment.CIRCLE_PR_NUMBER }}\
Expand Down Expand Up @@ -96,11 +96,6 @@ _commands:
(echo vcs_export && cat) >> lockfile.txt
sha256sum $PWD/lockfile.txt >> lockfile.txt

# Temp: Rolling transition to 24.04 leaves rosdep in 22.04 in the lurch
# This replaces main rosdep index with the last prior to release: Feb 28, 2024
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

apt-get update
rosdep update --rosdistro $ROS_DISTRO
dependencies=$(
Expand All @@ -109,6 +104,8 @@ _commands:
--ignore-src \
--skip-keys " \
slam_toolbox \
turtlebot3_gazebo \
gazebo_ros_pkgs \
" \
--verbose | \
awk '$1 ~ /^resolution\:/' | \
Expand Down
3 changes: 2 additions & 1 deletion .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,11 @@
"build": {
"dockerfile": "../Dockerfile",
"context": "..",
"target": "visualizer",
"target": "dever",
"cacheFrom": "ghcr.io/ros-planning/navigation2:main"
},
"runArgs": [
"--name=nav2"
// "--cap-add=SYS_PTRACE", // enable debugging, e.g. gdb
// "--ipc=host", // shared memory transport with host, e.g. rviz GUIs
// "--network=host", // network access to host interfaces, e.g. eth0
Expand Down
6 changes: 4 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ RUN apt-get update && \
ros-$ROS_DISTRO-rmw-fastrtps-cpp \
ros-$ROS_DISTRO-rmw-connextdds \
ros-$ROS_DISTRO-rmw-cyclonedds-cpp \
&& pip3 install \
&& pip3 install --break-system-packages \
fastcov \
git+https://github.com/ruffsl/colcon-cache.git@a937541bfc496c7a267db7ee9d6cceca61e470ca \
git+https://github.com/ruffsl/colcon-clean.git@a7f1074d1ebc1a54a6508625b117974f2672f2a9 \
Expand Down Expand Up @@ -101,6 +101,8 @@ RUN . $UNDERLAY_WS/install/setup.sh && \
--from-paths src \
--skip-keys " \
slam_toolbox \
turtlebot3_gazebo \
gazebo_ros_pkgs \
"\
--ignore-src \
&& rm -rf /var/lib/apt/lists/*
Expand Down Expand Up @@ -145,7 +147,7 @@ RUN apt-get update && \
bash-completion \
gdb \
wget && \
pip3 install \
pip3 install --break-system-packages \
bottle \
glances

Expand Down
3 changes: 3 additions & 0 deletions nav2_behavior_tree/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ find_package(nav2_util REQUIRED)

nav2_package()

# TODO: Delete after https://github.com/BehaviorTree/BehaviorTree.CPP/issues/811 is released
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File ticket with this commit as a reminder to check in on it, we don't do TODOs in this repo, they're forgotten about easily. Leave the comment though in-line (just not TODO).

add_compile_options(-Wno-shadow)

include_directories(
include
)
Expand Down
6 changes: 4 additions & 2 deletions nav2_map_server/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ find_package(std_msgs REQUIRED)
find_package(tf2 REQUIRED)
find_package(nav2_util REQUIRED)
find_package(GRAPHICSMAGICKCPP REQUIRED)
find_package(yaml-cpp REQUIRED)

nav2_package()

Expand Down Expand Up @@ -118,7 +119,8 @@ target_include_directories(${map_io_library_name} SYSTEM PRIVATE
${GRAPHICSMAGICKCPP_INCLUDE_DIRS})

target_link_libraries(${map_io_library_name}
${GRAPHICSMAGICKCPP_LIBRARIES})
${GRAPHICSMAGICKCPP_LIBRARIES}
yaml-cpp::yaml-cpp)

if(WIN32)
target_compile_definitions(${map_io_library_name} PRIVATE
Expand Down Expand Up @@ -160,5 +162,5 @@ ament_export_libraries(
${library_name}
${map_io_library_name}
)
ament_export_dependencies(${map_io_dependencies} ${map_server_dependencies})
ament_export_dependencies(${map_io_dependencies} ${map_server_dependencies} yaml-cpp)
ament_package()
9 changes: 6 additions & 3 deletions nav2_mppi_controller/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -64,9 +64,6 @@ if(COMPILER_SUPPORTS_FMA)
add_compile_options(-mfma)
endif()

# If building one the same hardware to be deployed on, try `-march=native`!
add_compile_options(-O3 -finline-limit=10000000 -ffp-contract=fast -ffast-math -mtune=generic)

add_library(mppi_controller SHARED
src/controller.cpp
src/optimizer.cpp
Expand All @@ -77,6 +74,9 @@ add_library(mppi_controller SHARED
src/noise_generator.cpp
)

# If building one the same hardware to be deployed on, try `-march=native`!
target_compile_options(mppi_controller PRIVATE -O3 -finline-limit=10000000 -ffp-contract=fast -ffast-math -mtune=generic)

add_library(mppi_critics SHARED
src/critics/obstacles_critic.cpp
src/critics/cost_critic.cpp
Expand All @@ -91,6 +91,9 @@ add_library(mppi_critics SHARED
src/critics/velocity_deadband_critic.cpp
)

# If building one the same hardware to be deployed on, try `-march=native`!
target_compile_options(mppi_critics PRIVATE -O3 -finline-limit=10000000 -ffp-contract=fast -ffast-math -mtune=generic)

set(libraries mppi_controller mppi_critics)

foreach(lib IN LISTS libraries)
Expand Down
8 changes: 6 additions & 2 deletions nav2_system_tests/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
cmake_minimum_required(VERSION 3.5)
project(nav2_system_tests)

if(NOT CMAKE_CXX_STANDARD)
set(CMAKE_CXX_STANDARD 23)
endif()

find_package(ament_cmake REQUIRED)
find_package(nav2_common REQUIRED)
find_package(rclcpp REQUIRED)
Expand All @@ -14,7 +18,7 @@ find_package(visualization_msgs REQUIRED)
find_package(geometry_msgs REQUIRED)
find_package(std_msgs REQUIRED)
find_package(tf2_geometry_msgs REQUIRED)
find_package(gazebo_ros_pkgs REQUIRED)
# find_package(gazebo_ros_pkgs REQUIRED)
find_package(nav2_amcl REQUIRED)
find_package(nav2_lifecycle_manager REQUIRED)
find_package(rclpy REQUIRED)
Expand All @@ -37,7 +41,7 @@ set(dependencies
nav2_amcl
nav2_lifecycle_manager
nav2_behavior_tree
gazebo_ros_pkgs
# gazebo_ros_pkgs
geometry_msgs
std_msgs
tf2_geometry_msgs
Expand Down
4 changes: 2 additions & 2 deletions nav2_system_tests/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
<build_depend>geometry_msgs</build_depend>
<build_depend>std_msgs</build_depend>
<build_depend>tf2_geometry_msgs</build_depend>
<build_depend>gazebo_ros_pkgs</build_depend>
<!-- <build_depend>gazebo_ros_pkgs</build_depend> -->
<build_depend>launch_ros</build_depend>
<build_depend>launch_testing</build_depend>
<build_depend>nav2_planner</build_depend>
Expand All @@ -48,7 +48,7 @@
<exec_depend>nav2_amcl</exec_depend>
<exec_depend>std_msgs</exec_depend>
<exec_depend>tf2_geometry_msgs</exec_depend>
<exec_depend>gazebo_ros_pkgs</exec_depend>
<!-- <exec_depend>gazebo_ros_pkgs</exec_depend> -->
<exec_depend>navigation2</exec_depend>
<exec_depend>lcov</exec_depend>
<exec_depend>robot_state_publisher</exec_depend>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,10 @@ class BehaviorTreeHandler
return false;
}

auto xml_string = std::string(
std::istreambuf_iterator<char>(xml_file),
std::istreambuf_iterator<char>());

std::stringstream buffer;
buffer << xml_file.rdbuf();
xml_file.close();
std::string xml_string = buffer.str();
// Create the blackboard that will be shared by all of the nodes in the tree
blackboard = BT::Blackboard::create();

Expand Down
4 changes: 0 additions & 4 deletions tools/underlay.repos
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,6 @@ repositories:
# type: git
# url: https://github.com/cra-ros-pkg/robot_localization.git
# version: ros2
# ros-simulation/gazebo_ros_pkgs:
# type: git
# url: https://github.com/ros-simulation/gazebo_ros_pkgs.git
# version: ros2
# ros2/geometry2:
# type: git
# url: https://github.com/ros2/geometry2.git
Expand Down