Skip to content

Shots tuning pre wcmp - #303

Merged
TaylerUva merged 6 commits into
mainfrom
shots-tuning-pre-wcmp
Apr 26, 2026
Merged

Shots tuning pre wcmp#303
TaylerUva merged 6 commits into
mainfrom
shots-tuning-pre-wcmp

Conversation

@Wu-Fan-529

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 26, 2026 02:07
@Wu-Fan-529
Wu-Fan-529 requested a review from a team as a code owner April 26, 2026 02:07

Copilot AI left a comment

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.

Pull request overview

Updates shooter tuning constants (flywheel RPM vs distance and hood angle vs distance) to refine shot performance ahead of competition.

Changes:

  • Adjusted flywheelSpeedMap RPM values and added additional close/mid-range calibration points.
  • Updated hoodAngleMap calibration points for 120/100/80-inch distances.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/main/java/frc/robot/constants/ConstRotors.java Tweaks flywheel RPM distance mapping; adds closer-distance calibration points.
src/main/java/frc/robot/constants/ConstMotion.java Adjusts hood-angle distance mapping for mid/close distances.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/java/frc/robot/constants/ConstRotors.java Outdated
Comment thread src/main/java/frc/robot/constants/ConstRotors.java
Comment thread src/main/java/frc/robot/constants/ConstMotion.java Outdated
Comment thread src/main/java/frc/robot/constants/ConstMotion.java Outdated
@TaylerUva
TaylerUva merged commit 6b7c430 into main Apr 26, 2026
2 checks passed
@TaylerUva
TaylerUva deleted the shots-tuning-pre-wcmp branch April 26, 2026 05:12
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