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
Added Jaco and mico meshes and urdfs #1
Conversation
modified: ada_srdf.xacro
modified: ada_srdf.xacro
modified: ada_urdf.xacro modified: kinova_common.xacro
modified: ada.urdf modified: ada_urdf.xacro modified: kinova_common.xacro
modified: ada.srdf modified: ada_srdf.xacro
deleted: README.txt modified: ada.urdf
|
||
find_package(catkin REQUIRED COMPONENTS | ||
urdf | ||
transmission_interface |
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.
Why is this necessary?
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.
If you look at ada.urdf.. it uses transmission to define the relationship between the actuators and the arm joints (defined as Simple Transmission type) ..
package.xml
Outdated
<package> | ||
<name>ada_description</name> | ||
<version>2.0.0</version> | ||
<description>Models for UW-ADA components</description> |
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.
Remove UW-
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.
done..
package.xml
Outdated
|
||
<license>TODO</license> | ||
|
||
<author email="xyz@uw.edu">XYZ</author> |
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.
You should put your name here. :)
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.
done..
[WIP] Set joint limits
new file: meshes/forque_kinova.dae modified: robots/ada_accessories.urdf
…cs/ada_description into robot_accessories
Robot accessories
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.
Do we want to include generated urdf files? Other X_description
don't have generated files.
@gilwoolee It seems most of the feedback is addressed. Can we merge this as it is and then fix further issues later? This is blocking setting up Travis CI of repos that depend on this repo. |
<scene> | ||
<instance_visual_scene url="#VisualSceneNode"/> | ||
</scene> | ||
</COLLADA> |
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.
Nit: newline.
|
||
<maintainer email="tapo@uw.edu">Tapomayukh Bhattacharjee</maintainer> | ||
|
||
<license>TODO</license> |
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.
Let's use BSD as other repos we have.
This initial PR is to add meshes and urdfs and srdfs to load the ADA model in rviz and support libada..