Skip to content
This repository has been archived by the owner on Nov 13, 2017. It is now read-only.

New getOnlyOneEndEffectorTip() function #262

Merged
merged 1 commit into from
Dec 8, 2015
Merged

New getOnlyOneEndEffectorTip() function #262

merged 1 commit into from
Dec 8, 2015

Conversation

davetcoleman
Copy link
Member

If you want to quickly solve a IK problem on the tip link of a joint model group using its end effector semantics, this is a helpful shortcut function

const moveit::core::LinkModel* getOnlyOneEndEffectorTip() const;

/** \brief If this group is an end-effector, specify the foot type (if any, may be NONE) */
void setEndEffectorFootType(const JointModelGroupFootType &end_effector_foot_type);
Copy link
Contributor

Choose a reason for hiding this comment

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

Am I missing something or is this additional function declared here without any implementation?

Copy link
Member Author

Choose a reason for hiding this comment

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

ooops, yes, that is part of my research code that leaked into this PR during a merge

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

@davetcoleman
Copy link
Member Author

ping can this be merged in?

@dornhege
Copy link
Contributor

dornhege commented Dec 4, 2015

+1 lgtm

I cannot test on jade, yet. So, you can merge yourself, if you can ensure it works correctly. Otherwise I'll get to it soon.

@davetcoleman
Copy link
Member Author

Thanks for looking at it. However, we've a rule to not merge our own PRs

@dornhege
Copy link
Contributor

dornhege commented Dec 8, 2015

Tested and works. Thanks.

dornhege added a commit that referenced this pull request Dec 8, 2015
@dornhege dornhege merged commit 42265de into moveit:jade-devel Dec 8, 2015
@davetcoleman davetcoleman mentioned this pull request Dec 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants