Skip to content

optimized & created path for right side#410

Open
VictorChen1-1 wants to merge 8 commits intomainfrom
change-pathplanner-autos-to-have-a-lower-risk-of-destroying-itself-due-to-early-activation
Open

optimized & created path for right side#410
VictorChen1-1 wants to merge 8 commits intomainfrom
change-pathplanner-autos-to-have-a-lower-risk-of-destroying-itself-due-to-early-activation

Conversation

@VictorChen1-1
Copy link
Copy Markdown
Contributor

Why are we doing this?

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

@VictorChen1-1 VictorChen1-1 requested a review from a team April 10, 2026 06:26
…hange-pathplanner-autos-to-have-a-lower-risk-of-destroying-itself-due-to-early-activation
@aschokking
Copy link
Copy Markdown
Contributor

For changes to auto like this, can you include a video of it working in the sim?

…k-of-destroying-itself-due-to-early-activation
…k-of-destroying-itself-due-to-early-activation
…k-of-destroying-itself-due-to-early-activation
…k-of-destroying-itself-due-to-early-activation
…k-of-destroying-itself-due-to-early-activation
…k-of-destroying-itself-due-to-early-activation
@VictorChen1-1
Copy link
Copy Markdown
Contributor Author

345t3r4yhhu345u.mp4

recording was started half a second late.

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.

3 participants