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

Obstacle layer don't subscribe laser scans #3059

Closed
PSotiriadis opened this issue Jul 4, 2022 · 3 comments
Closed

Obstacle layer don't subscribe laser scans #3059

PSotiriadis opened this issue Jul 4, 2022 · 3 comments

Comments

@PSotiriadis
Copy link

Hi
I want to report an issue with nav2_costmap_2d package. Using navigation2 with ROS2 Humble, local costmap is not indicated in RViz, like in screenshot below:

ObstacleLayerproblem

However, global costmap is working properly:

StaticLayerOK

So I thought that problem can be in obstacle layer and I took a look in navigation2->nav2_costmap_2d->plugins->obstacle_layer.cpp. I realized, that in line 226, we unsubscribe the laser scan subscriber, which is subscribed above in lines 224,225. I comment line 226 and local costmap is working now fine for me and is indicated in RViz normally:

ObstacleLayerOK

This is logical for me, but I don't know if it is desired from the authors of navigation2 (maybe it is desired to unsubscribe in this position, but forgot to subscribe later). I want only to highlight the problem!!!

PS: Same happens also later in line 255, where PointCloud2 subscriber is unsubscribed, but I didn't check if works or not. Please see also this position, because it can also produce a problem.

@padhupradheep
Copy link
Member

padhupradheep commented Jul 4, 2022

Already reported #3014 ! More info #3018

More info here https://discourse.ros.org/t/nav2-issues-with-humble-binaries-due-to-fast-dds-rmw-regression/26128/2

switch to cyclone dds and it works good

@SteveMacenski
Copy link
Member

SteveMacenski commented Jul 4, 2022

Indeed, working through it with Fast-DDS, but there’s the temp PR patch that can be applied to work with Fast-DDS if absolutely required, but I’d recommend just using Cyclone. Fast-DDS has a number of other issues at the moment as well.

@PSotiriadis
Copy link
Author

First, @SteveMacenski & @padhupradheep thanks a lot for the very fast response and sorry for the repetition of this issue. I switch to cyclone dds and everything works!!!

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

No branches or pull requests

3 participants