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

Fixed bug when there is no gazebo tag in urdf #12

Merged
merged 2 commits into from
Feb 10, 2022
Merged

Fixed bug when there is no gazebo tag in urdf #12

merged 2 commits into from
Feb 10, 2022

Conversation

GrmanRodriguez
Copy link
Contributor

In ergocub_gazebo_simulations we always expected some <gazebo> tags at the end of the URDF. This resulted in several bugs when using a URDF with no such tags.

This PR solves these bugs, and makes the library work with and without <gazebo> tags. Plus, they don't need to be at the end of the URDF (they will be written at the end in the output file however).

cc @traversaro

@traversaro
Copy link
Member

Thanks @GrmanRodriguez !

Copy link
Contributor

@AlexAntn AlexAntn left a comment

Choose a reason for hiding this comment

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

LGTM!

@AlexAntn AlexAntn merged commit af915e4 into master Feb 10, 2022
@AlexAntn AlexAntn deleted the fix_8 branch February 10, 2022 10:09
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.

The library requires that the model contains a <gazebo> element
3 participants