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

Add vel and acc scaling to computeCartesianPath in Humble #3051

Open
wants to merge 3 commits into
base: humble
Choose a base branch
from

Conversation

liamcarlos
Copy link

@liamcarlos liamcarlos commented Oct 30, 2024

Description

Hello, I'm back !

Made some more changes to properly integrate velocity and acceleration control to Cartesian Path Planning on the Humble branch of this repository. Also opened a PR on moveit/moveit_msgs to add these controls to the proper interface #184

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference
  • Document API changes relevant to the user in the MIGRATION.md notes
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

Add vel and acc scaling to computeCartesianPath on humble branch
Add vel and acc scaling to computeTimeStamps on humble branch
Copy link

mergify bot commented Oct 30, 2024

Please target the main branch for development, we will backport the changes to humble for you if approved and if they don't break API.

@liamcarlos
Copy link
Author

Hey @sjahr did this get merged to the humble branch? I'm not seeing any changes

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

Successfully merging this pull request may close these issues.

2 participants