Skip to content

Created timer for robot agitation - #61

Merged
jaymendo670-png merged 9 commits into
mainfrom
50-implement-intake-agitation
Jun 13, 2026
Merged

Created timer for robot agitation#61
jaymendo670-png merged 9 commits into
mainfrom
50-implement-intake-agitation

Conversation

@jaymendo670-png

Copy link
Copy Markdown
Contributor

No description provided.

Co-Authored-By: BrodyKarr <145169276+BrodyKarr@users.noreply.github.com>
Co-Authored-By: Justinpham17386769 <244249272+Justinpham17386769@users.noreply.github.com>
Co-Authored-By: jayden mendoza <244560677+jaymendo670-png@users.noreply.github.com>
@jaymendo670-png
jaymendo670-png requested a review from a team as a code owner June 13, 2026 04:12
@jaymendo670-png jaymendo670-png linked an issue Jun 13, 2026 that may be closed by this pull request

@TaylerUva TaylerUva left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Build failure

Comment thread src/main/java/frc/robot/commands/states/Shooting.java Outdated
Comment thread src/main/java/frc/robot/commands/states/Shooting.java Outdated
Comment thread src/main/java/frc/robot/commands/states/Shooting.java Outdated
Comment thread src/main/java/frc/robot/subsystems/Motion.java Outdated
Comment thread src/main/java/frc/robot/constants/ConstMotion.java Outdated

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

This PR adds an “intake pivot agitation” behavior during the Shooting command by periodically toggling the intake pivot angle on a timer, and centralizes the relevant timing/tolerance values in ConstMotion. It also removes an unused/duplicated “intake pivot at angle” helper from the Motion subsystem.

Changes:

  • Add INTAKE_PIVOT_TOLERANCE and INTAKE_PIVOT_AGITATION_TIME constants to ConstMotion.
  • Update Shooting to use a Timer to alternate the intake pivot between deploy/retract angles while shooting.
  • Remove the isIntakePivotAtAngle(...) helper and associated state from Motion.

Reviewed changes

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

File Description
src/main/java/frc/robot/subsystems/Motion.java Removes the intake-pivot “at angle” helper/state.
src/main/java/frc/robot/constants/ConstMotion.java Adds constants used to parameterize agitation timing/tolerance.
src/main/java/frc/robot/commands/states/Shooting.java Implements timer-driven intake pivot agitation during shooting and resets pivot/timer on end.

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

Comment thread src/main/java/frc/robot/commands/states/Shooting.java Outdated
Comment thread src/main/java/frc/robot/commands/states/Shooting.java Outdated
Comment thread src/main/java/frc/robot/commands/states/Shooting.java
@jaymendo670-png
jaymendo670-png merged commit 90f882f into main Jun 13, 2026
1 check passed
@jaymendo670-png
jaymendo670-png deleted the 50-implement-intake-agitation branch June 13, 2026 21:51
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.

Implement Intake Agitation

5 participants