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

Refactor/xacro launch generic elite description - generic elite arm robot description #63

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

AdarshKaran
Copy link

Added use_gazebo and use_arm_type arguments, changed urdf to give flexibility to use other elite arms by just adding meshes and config files, renamed filed

…nfig files, Refactored launch arguments-added use_gazebo arg in simulate_ec66.launch.py, restructured the packages
… the packages structure, modified urdf files to xacro using macros and yaml config files
…xibility to use other elite arms by just adding meshes and config files, renamed filed
Copy link
Member

@padhupradheep padhupradheep left a comment

Choose a reason for hiding this comment

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

Overall LGTM!

Could you please make a short video including the execution of MoveIt packages?

<xacro:include filename="$(find elite_description)/urdf/ec66_description.urdf" />

<!-- Import ec66 urdf file with use_gazebo set to true -->
<xacro:include filename="$(find elite_description)/urdf/ec66_description.urdf.xacro">
Copy link
Member

Choose a reason for hiding this comment

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

I think you need to update this.

2. made the elite arm xacro modular and compatible for both ec66 and cs66(yet to be added)
3. added initial positions for arm
@AdarshKaran AdarshKaran force-pushed the refactor/xacro-launch-generic-elite-description branch from f2509bc to b704112 Compare July 8, 2024 23:30
@AdarshKaran
Copy link
Author

i have made the updates to the elite packages anna. kindly let me know if it is okay. I will then add the functionality for cs66 arm too

@padhupradheep
Copy link
Member

But there is still an issue with the MoveIt package right?

@AdarshKaran
Copy link
Author

yes anna. But, the description files were updated.

@AdarshKaran AdarshKaran force-pushed the refactor/xacro-launch-generic-elite-description branch from 89a163d to 9312088 Compare July 18, 2024 18:35
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.

2 participants