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

Adds worlds code to develop #79

Merged
merged 5 commits into from
May 28, 2024
Merged

Adds worlds code to develop #79

merged 5 commits into from
May 28, 2024

Conversation

JakeWendling
Copy link
Collaborator

@JakeWendling JakeWendling commented May 26, 2024

PR Summary

PR Link: INSERT-LINK-HERE

Issue Link: INSERT-LINK-HERE

Description

Adds code from last day of worlds that wasnt pushed.

Reviewers


Changelog

  • Adds a bt node that signifies the end of the auton.
  • Adds parameter for the starting intake position.
  • Fixes runtime error where ros parameters were being registered multiple times by BT nodes.
  • changes various auton parameters in ros config

Reviewer Guide

  • Requires no input, just updating final values from last day of worlds

Testing

Automatic

  • Default tests

Manual

  • Should run correctly with hardware launch

Checklist

  • Confirmed all tests pass on a clean build
  • Added reviewers in Github
  • Posted PR Summary to Discord PR's Channel
  • Ran uncrustify on any modified C++ files
  • Ran Colcon Lint for any modified CMakeLists.txt or Package.xml

@JakeWendling JakeWendling merged commit 5bcf42a into develop May 28, 2024
1 check passed
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