Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Stretch robot and mobile manipulation planning #291

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

carlosquinterop
Copy link
Contributor

  • Creates a convenience class for the Stretch robot/planning and two test scripts.
  • Adds functions to the robot class to enable mobile manipulation planning. This feature relies on the existence of a kinematics solver plugin called base_manipulator_kinematics_plugin. See here.

Copy link
Collaborator

@ChamzasKonstantinos ChamzasKonstantinos left a comment

Choose a reason for hiding this comment

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

This implementation depends on a custom version of Moveit, that modifies the PlanarJoint.
We cannot merge it as it is now, since it breaks the code here: https://github.com/KavrakiLab/robowflex/runs/7571963014?check_suite_focus=true#step:3:2993

Maybe, remove the diff-drive feature? or make it dependent on a specific version of Moveit?

@@ -387,7 +387,7 @@ bool Robot::loadKinematics(const std::string &group_name, bool load_subgroups)
std::find(groups.begin(), groups.end(), name) == groups.end())
{
RBX_ERROR("No JMG or Kinematics defined for `%s`!", name);
return false;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Will this not create a problem?

scchow added a commit to scchow/robowflex that referenced this pull request May 5, 2023
- Based on PR: KavrakiLab#291
- Incorporating functions necessary to do mobile base + manipulator
  planning with robowflex.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants