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

Disabling transform timeout #306

Merged

Conversation

crdelsey
Copy link
Contributor

@crdelsey crdelsey commented Nov 9, 2018

Basic Info

Info Please fill out this column
Ticket(s) this addresses
Primary OS tested on Ubuntu 18.04
Primary platform tested on gazebo simulation turtlebot

Description of contribution in a few bullet points

  • Disabling transform timeout in AMCL. The current ROS2 transform code deadlocks when there is a transform timeout and use_sim_time is active. The timeout clock can't advance because the transform code is in a tight loop, preventing the node from spinning and servicing clock messages.
  • It seems it doesn't matter much that AMCL fails to process the laser scans due to missing transform, because occasionally the transform is available, and we can get a fix.

Future work that may be required in bullet points

Fix AMCL by re-introducing message filters, or finding some other wait to ensure a transform is available when there is laser scan data to process. #171

@SteveMacenski
Copy link
Member

Whenever short term solutions like this are implemented, I think its important to add a little comment explaining why and when to reimplement correctly as a TODO. This lets another reader debugging issues down the line find the root cause easily if this is forgotten about.

@crdelsey
Copy link
Contributor Author

crdelsey commented Nov 9, 2018

Good idea. I was thinking of this as removing a short term workaround rather than adding, so I didn't think to add the comment. I'll fix.

@SteveMacenski SteveMacenski self-requested a review November 9, 2018 22:58
@mhpanah mhpanah self-requested a review November 9, 2018 23:45
Copy link
Contributor

@mhpanah mhpanah left a comment

Choose a reason for hiding this comment

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

LGTM.

@crdelsey crdelsey merged commit ff249bd into ros-navigation:master Nov 12, 2018
@crdelsey crdelsey deleted the amcl-disabled-transform-timeout branch December 21, 2018 18:59
ghost pushed a commit to logivations/navigation2 that referenced this pull request Mar 7, 2022
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.

3 participants