-
Notifications
You must be signed in to change notification settings - Fork 6
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
QoS issues over scan topic with security enabled #12
Comments
We get the Here's what I know. This message comes from the message filter code in tf. It receives the scan messages, but ends up dropping them out of the buffer before it can transform them and pass them on to the stack. I think we'd see this message if amcl and costmap were not getting the right transforms or were not able to process them fast enough. There also were some race conditions in the most recent implementation of this code that @jacobperron fixed. There might still be some other problems |
I've opened a PR #13 to switch to using dashing release for everything but the newer SROS2 CLI, and if I spam the set initial pose using the script, it seems to work eventually. So I'm thinking this might be a discovery issue, where the subscribers for initial pose don't immediately find the publisher given the initial burst of discovery traffic, missing that adversized by rviz. So by using the ros2 cli to topic publish the pose, eventually upon enough tries in creating/destroying the cli node, the initial pose message finds its mark. |
Looks like the amcl and costmap nodes are not subscribing correctly, looking at it from the graph API: Working Unsecure demoUnlocalizedLocalizedFlaky Secure demoUnlocalized |
It seems that amcl is still having issues with dropping scan messages when security is enabled. That, or the tf lookup is having to look into the future for transforms between scan and baselink:
As of b0a41cb , I've updated the readme with some simple instructions, so please check if you can get the navigation planner to move when using the
secure.conf
. You can also revert 3a1562c if you'd like rebuild the dockerfile using osrf/ros2:nightly, but I'm not sure if rviz2 is fixed yet: ros-navigation/navigation2#1015 (comment) .The text was updated successfully, but these errors were encountered: