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

Suppress tf warnings #402

Merged
merged 1 commit into from
Nov 22, 2021
Merged

Suppress tf warnings #402

merged 1 commit into from
Nov 22, 2021

Conversation

caguero
Copy link
Contributor

@caguero caguero commented Nov 22, 2021

Now that this patch has been merged into the robot_localization package, we'll need to exercise this functionality.

Until there's a new ROS noetic release, we'll need to install robot_localization manually to observe this functionality working.

How to test it:

  1. First, compile robot_localization from source with our patch:
cd <your_vrx_ws>/src
git clone https://github.com/cra-ros-pkg/robot_localization.git
cd ..
catkin_make
  1. Launch vrx as usual:
roslaunch vrx_gazebo vrx.launch
  1. Launch our localization example
roslaunch wamv_gazebo localization_example.launch

You shouldn't see any warnings.

Signed-off-by: Carlos Agüero <[email protected]>
@M1chaelM M1chaelM self-requested a review November 22, 2021 20:58
Copy link
Collaborator

@M1chaelM M1chaelM left a comment

Choose a reason for hiding this comment

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

Thanks for making the change to robot_localization! I confirm this now runs without the repeated warnings.

@caguero caguero merged commit 82baedd into master Nov 22, 2021
@caguero caguero deleted the caguero/supress_tf_warnings branch January 5, 2023 21:21
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