-
Notifications
You must be signed in to change notification settings - Fork 938
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
Clarify "id" parameter in RobotState & collision world #1450
Clarify "id" parameter in RobotState & collision world #1450
Conversation
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.
LGTM. Thanks a lot.
362f5ee
to
6fa52e1
Compare
6fa52e1
to
632d3bb
Compare
Will be thankful for getting this merged soon, so the other PR becomes more readable. |
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.
LGTM as well, I'll merge when Travis passes.
My fault @rhaschke, I accidentally merged instead of squash-merging. Is there a good way to fix that on master? |
Done. There wasn't another commit yet. |
Description
This makes some parameters in the robot state and collision world clearer, as requested by @mlautman in #1060, in a separate PR as requested by @rhaschke in #1439. This PR changes nothing except the internal names.
Can be cherry-picked to kinetic and melodic.
Checklist