Skip to content

Add Findexer subsystem with motor control and feeding logic#37

Merged
ImSpxrsh merged 33 commits intodevelopfrom
26-findexer-intial-code
Feb 28, 2026
Merged

Add Findexer subsystem with motor control and feeding logic#37
ImSpxrsh merged 33 commits intodevelopfrom
26-findexer-intial-code

Conversation

@PillageDev
Copy link
Member

Add the Findexer subsystem that integrates with the shooter and turret subsystems.

Closes #26

NathanEdg and others added 28 commits January 28, 2026 16:26
…ebuilt into 13-turret-initial-code

# Conflicts:
#	src/main/java/frc/robot/subsystems/turret/Turret.java
#	src/main/java/frc/robot/subsystems/turret/TurretIOSensorInputs.java
# Conflicts:
#	src/main/java/frc/robot/CanID.java
#	src/main/java/frc/robot/RobotContainer.java
#	src/main/java/frc/robot/subsystems/vision/Vision.java
#	src/main/java/frc/robot/subsystems/vision/photon/Camera.java
…ic LowPassFilter, and added guardrails in the StateMachine
This reverts commit 1abef26.
This reverts commit 07ba2b5.
@PillageDev PillageDev linked an issue Feb 27, 2026 that may be closed by this pull request
Base automatically changed from 10-shooter-subsystem to 13-turret-initial-code February 27, 2026 18:35
Copy link
Contributor

@bjmcternan bjmcternan left a comment

Choose a reason for hiding this comment

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

There are too many other changes in here. Let's merge with develop first before continuing the PR

afranchuk
afranchuk previously approved these changes Feb 27, 2026
Copy link

@afranchuk afranchuk left a comment

Choose a reason for hiding this comment

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

143ba30 looks okay (I'm assuming that's the main part of the PR).

Base automatically changed from 13-turret-initial-code to develop February 28, 2026 19:26
@ImSpxrsh ImSpxrsh dismissed afranchuk’s stale review February 28, 2026 19:26

The base branch was changed.

@ImSpxrsh ImSpxrsh requested review from a team February 28, 2026 19:26
Copy link
Contributor

@bjmcternan bjmcternan left a comment

Choose a reason for hiding this comment

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

Looks good so far. Please add comments to the functions and variables

Copy link
Member

@ImSpxrsh ImSpxrsh left a comment

Choose a reason for hiding this comment

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

Looked through, looks good!

@ImSpxrsh ImSpxrsh merged commit 3556ffd into develop Feb 28, 2026
2 checks passed
@ImSpxrsh ImSpxrsh deleted the 26-findexer-intial-code branch February 28, 2026 21:39
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.

Findexer intial code

5 participants