Add kr6 r900-2 support#198
Conversation
gavanderhoorn
left a comment
There was a problem hiding this comment.
Seeing as this seems to reuse meshes in another package, we should state that dependency in the manifest.
Note reuse of kuka_kr10_support meshes
|
I added an exec_depend and amended the description in the manifest. |
|
@simonschmeisser: will you review this one? |
simonschmeisser
left a comment
There was a problem hiding this comment.
repeating mostly review comments @gavanderhoorn made yesterday on #162
…for xacro include
|
pushed a commit adressing the requests |
|
removed duplilcate tool0 |
|
@gavanderhoorn super friendly ping :) |
gavanderhoorn
left a comment
There was a problem hiding this comment.
Thanks 👍 Looks good.
And thanks for iterating.
Note: I've not tested this model/variant extensively. I'm trusting you have done that before submitting this PR.
|
You can see it "live" here: https://www.linkedin.com/posts/manfred-winter_binpicking-robotics-automation-activity-6848593022595162113-P3hr :) |
* Add KR 6 R900-2 support Note reuse of kuka_kr10_support meshes
* Add KR 6 R900-2 support Note reuse of kuka_kr10_support meshes
reuses identical kr10 r900-2 meshes