Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

Conversation

@IanTheEngineer
Copy link
Contributor

  • Migrates urdf_diagram.odg and urdf_diagram.png from robot_model pkg
    to this urdf pkg

  • Updates urdf_diagram for current repo structures and urls

  • Adds rendering from diagram in README.md

- Migrates urdf_diagram.odg and urdf_diagram.png from robot_model pkg
to this urdf pkg

- Updates urdf_diagram for current repo structures and urls

- Adds rendering from diagram in README.md
@IanTheEngineer
Copy link
Contributor Author

@clalancette here are the updates based on our discussion yesterday

Copy link
Collaborator

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks for the update @IanTheEngineer

@clalancette
Copy link
Collaborator

@sloretz Any objection to merging these binary data files into this git repository?

@clalancette clalancette merged commit e84c0ce into ros:melodic-devel May 8, 2020
@IanTheEngineer
Copy link
Contributor Author

@clalancette & @sloretz - I can't upload the new diagram to the ROS wiki (an admin is needed for image upload privileges i think?). Who should we ping to add the PR'd image to http://wiki.ros.org/urdf ?

@clalancette
Copy link
Collaborator

@clalancette & @sloretz - I can't upload the new diagram to the ROS wiki (an admin is needed for image upload privileges i think?). Who should we ping to add the PR'd image to http://wiki.ros.org/urdf ?

I've uploaded it now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants