-
Notifications
You must be signed in to change notification settings - Fork 131
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
Feature/issue 22 #23
Feature/issue 22 #23
Conversation
… test for old PR2 transmission types. test/calvin: Added in generated Calvin URDF provided by v4hn test/romeo: Reformatted to match Calvin test
…<gazebo reference="..."/>)
…n the same robot. test: Added in specific test for tranmsissions. Have it print the type. @todo: Make this a unittest.
@@ -329,7 +329,7 @@ def __init__(self, name=None, visual=None, inertial=None, collision=None, origin | |||
]) | |||
|
|||
|
|||
class Transmission(xmlr.Object): | |||
class PR2Transmission(xmlr.Object): |
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.
Is this PR2 specific? I am a bit confused about this. URDF does not have PR2-specific tags.
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.
See http://wiki.ros.org/urdf/XML/Transmission
The [old] transmission element is an extension to the URDF robot description model.
This extension is used for the PR2 robot.
It looks like they didn't want to break the old usage of the transmission elements.
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.
Thank you for the clarification!
Use c++11 instead of boost, bump version to 1.0.0
Bugfix for #22 described by v4hn. Adds duck-typed parsing for tags so that either the new transmission types can be parse or the old PR2 types can be parsed transparently.
Added in
calvin
robot description, along with test parsing and generated patch, as well as a test parsing both transmission types in the same robot.(After reviewing this old code, I realized that I really need to document this and clean it up so it's a little more understandable!)