Skip to content
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

Improved mapper node to perform localization with a given map and a specified initial pose. #33

Open
wants to merge 1 commit into
base: indigo_devel
Choose a base branch
from

Conversation

carlosmccosta
Copy link

  • Fixed loading of point clouds with colors and normals.
  • Added publishing of filtered and full sensor registered
    sensor_msgs::PointCloud2 msgs
  • Added publishing of geometry_msgs::PoseStamped in a new configurable
    frame_id (baseFrame)
  • Added option to invert tf (useful to bypass limitation of tf being a
    tree)
  • Added setup of initial pose from parameter server at startup
  • Added setup of map using path from parameter server at startup
  • Changed most ROS_INFO to ROS_DEBUG to not cause unnecessary console
    information overload
  • Added conversion between geometry_msgs::Pose and Eigen matrix
  • Added timeout parameter to
    PointMatcher_ros::transformListenerToEigenMatrix

Tested in Ubuntu 14.04 with ROS Indigo for comparison with DRL system:
https://github.com/carlosmccosta/dynamic_robot_localization_tests/blob/hydro-devel/launch/environments/asl/bags/ethzasl_kinect_dataset_high_complexity_slow_fly_movement.launch

roslaunch dynamic_robot_localization_tests ethzasl_kinect_dataset_high_complexity_slow_fly_movement.launch use_dynamic_robot_localization:=0 use_ethzasl_icp_mapper:=1

Note:
Requires pull request
norlab-ulaval/libpointmatcher#103

loading of point clouds with colors and normals.

- Added publishing of filtered and full sensor registered
sensor_msgs::PointCloud2 msgs
- Added publishing of geometry_msgs::PoseStamped in a new configurable
frame_id (baseFrame)
- Added option to invert tf (useful to bypass limitation of tf being a
tree)
- Added setup of initial pose from parameter server at startup
- Added setup of map using path from parameter server at startup
- Changed most ROS_INFO to ROS_DEBUG to not cause unnecessary console
information overload
- Added conversion between geometry_msgs::Pose and Eigen matrix
- Added timeout parameter to
PointMatcher_ros::transformListenerToEigenMatrix
@pomerlef
Copy link
Contributor

ok, that one will take me a bit more time to review. I can at least say that the reported error is not caused by your patch, but from a configuration problem on our Jenkins server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants