Skip to content

Hermite Splines#271

Draft
JohnGilb wants to merge 12 commits intomainfrom
feature/Hermite
Draft

Hermite Splines#271
JohnGilb wants to merge 12 commits intomainfrom
feature/Hermite

Conversation

@JohnGilb
Copy link
Copy Markdown
Contributor

@JohnGilb JohnGilb commented Mar 8, 2025

Why are we doing this?

Was thinking about a different kind of spline follower, pathing along the spline directly rather than creating several intermediate points and re-using the SimpleSwerveTrajectory. Overall felt a little easier to tune.

Is also about 80% of the way to hooking up the NeoTrellis buttons to scoring/collection command groups using said splines, which should allow for a relatively simple drop-in of a more advanced source of splines (e.g. the Orin)

Asana task URL:

Whats changing?

Questions/notes for reviewers

How this was tested

  • tested on robot
  • tested in simulator
  • unit tests added

Video/screenshots (from simulator or live robot)


PR feedback legend

Symbol Meaning
⭐ ⭐ ⭐ must be addressed
⭐ ⭐ should be addressed
something to consider, a good idea

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there an off the shelf library we could use for this math?

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