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

Added a helper function to extract joint groups. #259

Merged
merged 1 commit into from
Jan 22, 2016

Conversation

psigen
Copy link
Member

@psigen psigen commented Jan 22, 2016

This commit creates a helper function that copies OpenRAVE's
hardcoded table of joint derivative names and adds an accessor
function for retrieving them from a configuration specification.

This allows users to directly query the n-th derivative group of
a trajectory without having to iterate through other derivatives.

I could not think of a cleaner way to do this, but at least this
avoid having the values hard-coded in multiple places.

Note that this function only handles joint value derivatives. A
corresponding function for affine derivatives could be added in the
future.

This commit creates a helper function that copies OpenRAVE's
hardcoded table of joint derivative names and adds an accessor
function for retrieving them from a configuration specification.

This allows users to directly query the n-th derivative group of
a trajectory without having to iterate through other derivatives.

I could not think of a cleaner way to do this, but at least this
avoid having the values hard-coded in multiple places.

Note that this function only handles joint value derivatives.  A
corresponding function for affine derivatives could be added in the
future.
@mkoval
Copy link
Member

mkoval commented Jan 22, 2016

It's sad that this is necessary, but it is a very useful function to have.

mkoval added a commit that referenced this pull request Jan 22, 2016
…iveGroup

Added a helper function to extract joint groups.
@mkoval mkoval merged commit cde6d1f into master Jan 22, 2016
@mkoval mkoval deleted the feature/GetJointDerivativeGroup branch January 22, 2016 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants