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 built in path generation #13

Open
wants to merge 5 commits into
base: development
Choose a base branch
from

Conversation

Ryan4253
Copy link
Collaborator

Description
Adds path generation through bezier curves / line smoother

Related Issue
#2

Motivation and Context
I hate pathplanner. I can create less densely populated paths which makes pure pursuit more performant

How Has This Been Tested?
Outputted waypoints and plotted on excel
Should be tested on a v5 brain by outputting generated coordinates

Screenshots (if appropriate):
image
image

Notes
Probably will have a merge conflict associated with the bug fixes, but I've added those in the commit
All of the code can probably be constexpr'd which means that all the path will be generated at compile time

@jazonshou
Copy link
Owner

  1. My original plan/idea was to keep pathing & all the path generator stuff in lib4253. Then, we can use lib4253 to generate paths for HolonomicLib.
  2. fix the merge conflict

ill look at this more carefully later but I still think we should dedicate pathing to lib4253.

@Ryan4253
Copy link
Collaborator Author

Also @Yessir120 can you set this PR to a draft for now? I still need to add comments for the headers.

@jazonshou
Copy link
Owner

@Ryan4253 When do you think you can finish this?

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.

None yet

2 participants