Skip to content

Make Turret ╰(*°▽°*)╯#2

Merged
TaylerUva merged 11 commits intomainfrom
1-create-a-new-turret
Nov 16, 2025
Merged

Make Turret ╰(*°▽°*)╯#2
TaylerUva merged 11 commits intomainfrom
1-create-a-new-turret

Conversation

@S0L0GUY
Copy link
Copy Markdown
Contributor

@S0L0GUY S0L0GUY commented Nov 15, 2025

No description provided.

@S0L0GUY S0L0GUY linked an issue Nov 15, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Member

@TaylerUva TaylerUva left a comment

Choose a reason for hiding this comment

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

Lets remove pivot from places since its redundant

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 implements turret subsystem functionality by adding motor control for ball shooter, turret pivot, and hood mechanisms.

  • Added Rotors subsystem with ball shooter motor control
  • Added Motion subsystem with turret pivot and hood angle control using MotionMagic
  • Configured motor parameters including PID values, soft limits, and current limits

Reviewed Changes

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

Show a summary per file
File Description
Rotors.java Implements ball shooter motor control with velocity monitoring
Motion.java Implements turret pivot and hood angle control with position tracking and coast/brake mode switching
ConstRotors.java Defines TalonFX configuration for ball shooter motor
ConstMotion.java Defines TalonFX configurations for turret and hood motors
RobotMap.java Adds CAN IDs for ball shooter, turret pivot, and hood motors

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

@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@S0L0GUY S0L0GUY requested a review from a team as a code owner November 16, 2025 00:27
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings November 16, 2025 00:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

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


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

Copilot AI review requested due to automatic review settings November 16, 2025 00:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

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


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

Copilot AI review requested due to automatic review settings November 16, 2025 02:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

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


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

@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
Copilot AI review requested due to automatic review settings November 16, 2025 02:55
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

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


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

@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@FRCTeam3255 FRCTeam3255 deleted a comment from Copilot AI Nov 16, 2025
@TaylerUva TaylerUva enabled auto-merge (squash) November 16, 2025 03:03
@TaylerUva TaylerUva merged commit 6e1cea6 into main Nov 16, 2025
1 check passed
@TaylerUva TaylerUva deleted the 1-create-a-new-turret branch November 16, 2025 03:03
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 a New Turret

3 participants