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

non unified urdf.xacro #194

Open
guihomework opened this issue Jan 21, 2015 · 1 comment
Open

non unified urdf.xacro #194

guihomework opened this issue Jan 21, 2015 · 1 comment

Comments

@guihomework
Copy link
Contributor

With the addition of the lite hand, again, new hand specific copies of palm, forearm etc.. appeared in the sr_description whereas some other xacro macros remained generic but got a specific is_lite parameter.
Last year I worked hard to try unify the files for different types of hand in order to never have copy paste to do in all the files in case one needs to fix a small thing in one element. With several files, there are redundant information and updates might be forgotten in all instances of the files.

recent xacro 1.9.3 ros/xacro@ee442c9 (soon available in upstream ros indigo), includes default parameters for xacro:macro.
This way is_lite:=0 could be added to palm and forearm macros, which will not require any change to the standard hands using the macro as is, without adding the is_lite parameter, and the new hand lite could use the same macro specifying is_lite=1 to override the default. The same could be done for a lot of other parameters (creating default standard hand settings and cleaning up the long list of parameters for optional values)

Isn't this the better way to go to keep the sr_description maintainable ?

@toliver
Copy link
Contributor

toliver commented Jan 21, 2015

It certainly is a better way. We'll do these changes as soon as we can.

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

No branches or pull requests

2 participants