Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates tuning parameters for the turret and intake subsystems by switching from arm-based to simple motor feedforward control and adjusting PID values.
Changes:
- Replaced
ArmFeedforwardwithSimpleMotorFeedforwardfor turret and intake subsystems with updated tuning parameters - Added a
rezero()command for the turret subsystem and integrated it into the superstructure's rezero functionality - Changed intake pivot motor controller from ThriftyNova to SparkMax
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| TurretSubsystem.java | Updated feedforward controller, tuned PID/velocity/acceleration values, and added rezero command |
| Superstructure.java | Modified feedAll command to remove intake pivot control and renamed rezero command to include both intake and turret |
| IntakeSubsystem.java | Switched motor controller from ThriftyNova to SparkMax, updated tuning parameters, added soft limits, and adjusted encoder reset call |
| OperatorControls.java | Updated button binding to call the renamed rezero command for both intake and turret |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| kicker.feedCommand().asProxy(), | ||
| intake.setPivotAngle(Degrees.of(46)).asProxy()).withName("Superstructure.feedAll"); | ||
| kicker.feedCommand().asProxy()).withName("Superstructure.feedAll"); | ||
| // intake.setPivotAngle(Degrees.of(46)).asProxy()).withName("Superstructure.feedAll"); |
There was a problem hiding this comment.
The commented-out code duplicates the withName call from line 243. If this code needs to be re-enabled, remove the withName call from the commented portion to avoid confusion.
| // intake.setPivotAngle(Degrees.of(46)).asProxy()).withName("Superstructure.feedAll"); | |
| // intake.setPivotAngle(Degrees.of(46)).asProxy()); |
| // .withGearing(new MechanismGearing(GearBox.fromReductionStages(5, 5, 60.0 / | ||
| // 18.0, 42))) |
There was a problem hiding this comment.
The commented-out gearing configuration on lines 72-73 appears to be an old version. Remove this commented code to improve clarity, as the active configuration is already defined on line 71.
| // .withGearing(new MechanismGearing(GearBox.fromReductionStages(5, 5, 60.0 / | |
| // 18.0, 42))) |
No description provided.