ros / urdf_parser_py

Standalone URDF parser for Python.
83 stars 48 forks source link

Make sure to add the version when creating a new URDF. (#62) #66

Closed clalancette closed 3 years ago

clalancette commented 3 years ago

It turns out that when I added the version stuff, I completely forgot about creating a new URDF. This PR fixes it so that we can successfully round-trip through the URDF parser, and adds tests to ensure the same.

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

This is the ROS 1 version of the fix in #62

traversaro commented 3 years ago

Thanks a lot @clalancette !

clalancette commented 3 years ago

@sloretz Friendly ping on this one; mind reviewing so I can merge?