-
Notifications
You must be signed in to change notification settings - Fork 5
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
make shadowhand_motor.srdf more generic #5
Comments
Wouhou, did it for the srdf, using xacro. Can theoretically handle any hand (any number of finger/is_lite) + prefix in joint_names but the problem is now on yaml files. They also would need the prefix argument to be passed as well as "if" in case some fingers are not there etc... |
what about having a python script utility which parses a default yaml file and pushes the properly aggregated parameters to the parameter server? It should be easier to maintain and more flexible. |
I am not sure I understand your idea : Now the xacro for SRDF works like this and can be called from a lunch file |
Are you talking about the yaml files to define the controllers? |
No moveit files
|
I think Ugo's proposal makes sense. The default launchfiles created by MoveIt simply load these yaml files to the parameter server (e.g. in planning_context.launch). They could be modified to call instead a python script that reads the default yaml file, adds prefixes/removes undesired joints/fingers, and then loads the result to the parameter server. |
So instead of generating the correct yaml, you want to use the full yaml, parse it, and load correct parameters on the server. I thought it would be cleaner to read a template file (that could also be yaml i you dont' like the yaml embeded in xml (see email)), except with nice valid yaml language that the python parser could find in the template instead of finding what exists and should not be there. The only advantage I see, is that you don't have to invent a new macro language. However a new macro language for yaml could be more widely used later on... I guess this solution could work nice for the yaml but a similar technique might be difficult to use for the srdf or did you plan to do this the same way ? using the full srdf, parsing and cleaning what is unnecessary and prefixing everything (prefixing is at various places, non only joints), and collision pairs are quite complex to handle too. I am double checking the generated collisions (500 pairs to check) and this srdf generator should be ok fully ready. |
(from sr_hand_kinematics)
The text was updated successfully, but these errors were encountered: