-
Notifications
You must be signed in to change notification settings - Fork 333
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 tf_prefix based on namespace #1060
Improved tf_prefix based on namespace #1060
Conversation
…f_prefix when not there yet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look fine to me!
Thank you!
Co-authored-by: Christoph Fröhlich <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look at the failing tests
What is the problem with adding this fix? This issue has been going on almost for a year, and is it just a matter of a few lines of code. |
Acc to my last comment, tests were failing. As this PR is locked for maintainers, I can't push something to it or merge the current master into it.
Maybe someone wants to fix this on top of this PR and open a new one. |
Fixed with #1420 |
Solves #1048.
When using robot namespace as
tf_prefix
, it now removes any starting '/' and only adds an ending '/' if not present already.Starting with '/' is not common in TF2 and showed compatibility issues with for instance
robot_localization
.