Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cj/krakendrive #34

Merged
merged 4 commits into from
May 28, 2024
Merged

Cj/krakendrive #34

merged 4 commits into from
May 28, 2024

Conversation

ApplePiRobot
Copy link
Contributor

@ApplePiRobot ApplePiRobot commented May 19, 2024

Description

Tuned PIDs and Current limits for new Kraken X60 drive motors and added a counter auto for aces high for CT champs.
Wheel offsets will need to be rest next time in shop.

Type of change

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Bugfix (non-breaking change which fixes an issue, add DEF # above in description)
  • Environment change (changes to developer environment or CI configuration)
  • Documentation Update (if none of the other choices apply)

How Has This Been Tested?

Tested in shop and at CT Champs

Author Checklist:

  • I have performed a self-review of my code
  • I have added 1-2 reviewers
  • I have @mentioned relevant non-reviewers as an FYI
  • I have included a detailed description of my changes and their motivations

@@ -14,7 +14,7 @@ public class SwerveModule {
// Back left, back right, front left, front right convention.
// Calibration procedure: set all offsets to 0.0, redeploy, align all wheels to face forwards,
// use CANCoder reading as offset, add 180.0 if wheel drives backwards.
private static final double[] STEER_WHEEL_ZERO_OFFSET_DEGREES = {298.04, 260.33, 161.37, -0.44};
private static final double[] STEER_WHEEL_ZERO_OFFSET_DEGREES = {298.04, 260.33, 161.37, -0.44}; //adjust wheel offsets.
Copy link
Contributor

Choose a reason for hiding this comment

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

Retune the steer offsets before merging

Copy link
Contributor

@gordonbchen gordonbchen left a comment

Choose a reason for hiding this comment

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

👍

@CJc318 CJc318 merged commit b868948 into main May 28, 2024
1 check passed
@CJc318 CJc318 deleted the cj/krakendrive branch May 28, 2024 22:54
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