-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ROS2 port of cartographer_ros (library & executables) #1557
Comments
Hey, |
It seems that the install fails. I created an issue #1579. |
@jediofgever Work in progress. Not yet ready to build. @seasony-vp This is unrelated to this issue. You are referring to the old port made by OSRF, we plan to make a new one here. |
Dear @MichaelGrupp
Do you have any updates regarding this? |
I didn't have the time for continuing the WIP state recently. But I will update here once there's something new. |
PR for rolling/galactic here #1622 |
Components
cartographer_ros/CMakeLists.txt
)cartographer_ros/cartographer_ros/CMakeLists.txt
)The library cmake file uses a glob filter to get all
.cc
files. This could be changed for the period of porting, to allow porting only a subset of files at a time (in correct dependency order of course).Regarding executables I would say the first goal should be to port
node_main.cc
, thenoffline_node_main.cc
and then all the remaining tools. Files that don't build yet should be excluded from the build temporarily, by commenting out in the cmake file.How to test locally with Docker
Can be done in addition to building with
colcon
in your personal workspace.This is how we would test before merging, so it makes also sense to test it like this locally.
Once #1543 and #1542 are merged, you can activate the build of cartographer_ros in
Dockerfile.foxy
and inscripts/install_debs.sh
(see TODO comments in these files). Then runto build your branch (dashing can be done the same way).
As written also in #1536, you can have a look also at the outdated old port on the
ros2-dashing-1.0.0
branch.The text was updated successfully, but these errors were encountered: