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

Extend frenet_optimal_trajectory to support more scenarios #1114

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

Conversation

Aglargil
Copy link

@Aglargil Aglargil commented Dec 24, 2024

Reference issue

fix #1073

What does this implement/fix?

  • Add CartesianFrenetConverter: A class for converting states between Cartesian and Frenet coordinate systems
  • Add third derivative and curvature rate calculations to CubicSpline
  • Add LowSpeed/HighSpeed LateralMovementStrategy, add MergingAndStopping/VelocityKeeping LongitudinalMovementStrategy
  • Fix : s_d was used as the acceleration, but since s_d only represents the longitudinal acceleration, the CartesianFrenetConverter is now used to compute the acceleration a consistently

Additional information

  • Since I’m unsure which formatting tool to use, I used ruff as mentioned in this link. If there’s a standard formatting method for the project, please let me know, and I’ll update it right away.

CheckList

  • Did you add an unittest for your new example or defect fix?
  • Did you add documents for your new example? (Already have)
  • All CIs are green? (You can check it after submitting)

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.

Extend frenet_optimal_trajectory to support more scenarios
1 participant