Skip to content

Base autos (//▽//) - #81

Merged
TaylerUva merged 6 commits into
mainfrom
base-autos
Aug 9, 2026
Merged

Base autos (//▽//)#81
TaylerUva merged 6 commits into
mainfrom
base-autos

Conversation

@Wu-Fan-529

Copy link
Copy Markdown
Member

No description provided.

@Wu-Fan-529
Wu-Fan-529 requested a review from a team as a code owner August 8, 2026 22:32
Copilot AI lite review requested due to automatic review settings August 8, 2026 22:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR expands the robot’s Choreo autonomous library by adding multiple “Preview” trajectories and regenerating the associated Java constants/trajectory registry so the code can reference these paths consistently at runtime.

Changes:

  • Regenerated ChoreoVars with additional Choreo variables (typed units) and a larger set of named field poses.
  • Regenerated ChoreoTraj with new trajectory constants and updated ALL_TRAJECTORIES lookup entries.
  • Added/updated multiple Choreo .traj files and updated _all_paths.chor variable/pose definitions (including bumper dimension updates).

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/main/java/frc/robot/constants/ChoreoVars.java Adds generated Choreo variables (typed units) and many named poses used by trajectories.
src/main/java/frc/robot/constants/ChoreoTraj.java Adds new generated trajectory constants and registers them in ALL_TRAJECTORIES.
src/main/deploy/choreo/PreviewSecondDSideTrenchNeutral.traj Adds a new D-side trench-to-neutral preview trajectory definition.
src/main/deploy/choreo/PreviewPreloadOnly.traj Renames the path and updates parameters for a preload-only trajectory placeholder.
src/main/deploy/choreo/PreviewPreloadAndBackup.traj Adds a new preview preload-and-backup trajectory definition.
src/main/deploy/choreo/PreviewOSidePreloadAndBackup.traj Adds a new O-side preview preload-and-backup trajectory definition.
src/main/deploy/choreo/PreviewFirstOSideTrenchNeutral.traj Adds a new O-side trench-to-neutral preview trajectory definition.
src/main/deploy/choreo/PreviewFirstDSideTrenchNeutral.traj Adds a new D-side trench-to-neutral preview trajectory definition.
src/main/deploy/choreo/AutoPIDTuning.traj Updates bumper dimensions in the PID tuning trajectory config.
src/main/deploy/choreo/_all_paths.chor Adds shared expressions/poses and updates bumper dimensions used across Choreo paths.
Suppressed comments (1)

src/main/java/frc/robot/constants/ChoreoVars.java:18

  • New constants in this file use lower_snake_case (e.g., against_hub), which conflicts with the project’s established constants naming convention (SCREAMING_SNAKE_CASE) for fields in constants/. Consider renaming these generated constants (or adjusting Choreo codegen settings) so Java code can reference them in the expected style.
        public static final Pose2d against_hub = new Pose2d(3.5825274, 4.0294271, Rotation2d.fromRadians(0));

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/main/java/frc/robot/constants/ChoreoVars.java Outdated
Comment thread src/main/java/frc/robot/constants/ChoreoVars.java
Comment thread src/main/java/frc/robot/constants/ChoreoTraj.java Outdated
Wu-Fan-529 and others added 4 commits August 8, 2026 16:12
Co-Authored-By: BrodyKarr <145169276+BrodyKarr@users.noreply.github.com>
Co-Authored-By: jayden mendoza <244560677+jaymendo670-png@users.noreply.github.com>
Co-Authored-By: Justinpham17386769 <244249272+Justinpham17386769@users.noreply.github.com>
Co-Authored-By: Tej shah <187054380+Tejshah88@users.noreply.github.com>
Co-Authored-By: lameesnotlame <258483139+lameesnotlame@users.noreply.github.com>
@Wu-Fan-529 Wu-Fan-529 linked an issue Aug 9, 2026 that may be closed by this pull request
@TaylerUva
TaylerUva enabled auto-merge (squash) August 9, 2026 01:10
@TaylerUva
TaylerUva merged commit e238be8 into main Aug 9, 2026
1 check passed
@TaylerUva
TaylerUva deleted the base-autos branch August 9, 2026 01:11
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.

Create auto paths

3 participants