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

Add ar2010 support #396

Conversation

Doomerdinger
Copy link
Contributor

Added support for the AR2010.

Requires #378.

Alec Tiefenthal added 2 commits December 7, 2020 13:43
Made robots use materials from that package so they do not conflict when using multiple arms.
@gavanderhoorn
Copy link
Member

gavanderhoorn commented May 15, 2021

Similar to #395, it appears this model has been constructed such that all joints rotate around the Z axis.

Please see my #395 (comment) there.

The _macro.xacro also contains some gazebo references, but doesn't include inertial nor dynamics elements. Which would make it not work with Gazebo in its current state afaik.

Could you please remove the Gazebo-isms?

As in #395: I'm willing to do it, but I wanted to ask you first.

@Doomerdinger
Copy link
Contributor Author

@gavanderhoorn if you have time, you mentioned you could do what is necessary to bring this up to snuff to get it into the main branch?

I've had slim to no time to work on this stuff in quite some time.

@gavanderhoorn
Copy link
Member

I'll try to get to it. Thanks for letting us know @Doomerdinger.

@gavanderhoorn
Copy link
Member

Replacement: #480.

Thanks for the initial work @Doomerdinger 👍

gavanderhoorn added a commit that referenced this pull request Mar 23, 2022
Squashed commits:

* Added AR2010 support.
* ar2010: update minimum CMake version to avoid warnings on Noetic
* ar2010: add XML preambles where they were missing
* ar2010: tabs to spaces
* ar2010: remove Gazebo-isms
* ar2010: use correct "no-version" version
* ar2010: correctly capitalise series name
* ar2010: tests should have variant suffix
* ar2010: add note about all joints rotating around Z+

Co-authored-by: Alec Tiefenthal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants