Skip to content

2025 comp dev #14

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.

Sign up for GitHub

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

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

2025 comp dev #14

redto0 wants to merge 4 commits into from

Conversation

@redto0
Copy link
Member

@redto0 redto0 commented Dec 23, 2025

Changes

  • commented out the publish_stop_command()
  • idea is we can keep going left.
  • made get spline std::optional(). This allows us to reject an empty path and continue with the current one
  • fixed a bug where min speed was actually just max speed
  • removed DDA from the spline generation.

TODO

  • make a decision on the status of You Only Look Left, as this was a camera fix, it might not be ideal behavor for the kart in a GPS based system.
  • Make sure the est of the changes are ideal for the GPS kart.

redto0 and others added 3 commits May 3, 2025 17:53
WIP Updated Spline to not upate if brokern and added better no point …
2e5e798 
…support

Signed-off-by: redto0 <aboccacc@gmail.com>
@redto0
Bug fix!
4ddde60 
Now with more speed!

Signed-off-by: redto0 <aboccacc@umich.edu>
@redto0
minor comments
bf72b31
@redto0 redto0 self-assigned this Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Redderpears Redderpears Awaiting requested review from Redderpears

@MarkPopkov MarkPopkov Awaiting requested review from MarkPopkov

@estickel estickel Awaiting requested review from estickel

Assignees

@redto0 redto0

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@redto0