Skip to content

Reset hood angle when boots up (〃'▽'〃) - #302

Merged
TaylerUva merged 2 commits into
mainfrom
reeset-hood-angle-when-boots-up
Apr 25, 2026
Merged

Reset hood angle when boots up (〃'▽'〃) #302
TaylerUva merged 2 commits into
mainfrom
reeset-hood-angle-when-boots-up

Conversation

@Wu-Fan-529

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 25, 2026 19:00
@Wu-Fan-529
Wu-Fan-529 requested a review from a team as a code owner April 25, 2026 19:00
@Wu-Fan-529 Wu-Fan-529 changed the title Reset hood angle when boots up Reset hood angle when boots up (〃'▽'〃) Apr 25, 2026

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

Resets the hood’s encoder position on robot startup and updates the “none” hood angle calibration constant.

Changes:

  • Initialize hood TalonFX position to HOOD_NONE_ANGLE during Motion subsystem construction.
  • Update ConstMotion.HOOD_NONE_ANGLE from 1.6° to 3.2°.

Reviewed changes

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

File Description
src/main/java/frc/robot/subsystems/Motion.java Sets hood sensor position at subsystem initialization to establish a startup reference angle.
src/main/java/frc/robot/constants/ConstMotion.java Adjusts the calibrated “none” hood angle constant used as the reference angle.

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

Comment thread src/main/java/frc/robot/constants/ConstMotion.java
Comment thread src/main/java/frc/robot/subsystems/Motion.java
@TaylerUva
TaylerUva merged commit 5bb64bf into main Apr 25, 2026
6 checks passed
@TaylerUva
TaylerUva deleted the reeset-hood-angle-when-boots-up branch April 25, 2026 20:44
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