commit 3f1bbf5b68237d6dad7afa4a7ab68978c8e23f58
author James Kuszmaul <jabukuszmaul+collab@gmail.com> Sat Jun 22 15:57:44 2024 -0700
committer James Kuszmaul <jabukuszmaul+collab@gmail.com> Sat Aug 10 11:58:18 2024 -0700
tree 64dc04e4382f03ee4d99dde818b68f8f9af65803
parent df6b1685342014be6fe2cac27f58ee7f08b556ca

Add motor gear ratios to y2024_swerve wpilib

Change-Id: I2e0e54dd2cb5d131c70d26e2189553244ed6fb1a
Signed-off-by: James Kuszmaul <jabukuszmaul+collab@gmail.com>

y2024_swerve/constants.h

diff --git a/y2024_swerve/constants.h b/y2024_swerve/constants.h
index cf6c183..b5a89dd 100644
--- a/y2024_swerve/constants.h
+++ b/y2024_swerve/constants.h
@@ -24,12 +24,12 @@
   static constexpr double kDrivetrainStatorCurrentLimit() { return 35.0; }
   static constexpr double kDrivetrainSupplyCurrentLimit() { return 60.0; }
 
-  // TODO (maxwell): Make this the real value
-  static constexpr double kFollowerWheelCountsPerRevolution() { return 512.0; }
-  static constexpr double kFollowerWheelEncoderRatio() { return 1.0; }
-  static constexpr double kFollowerWheelRadius() { return 3.25 / 2 * 0.0254; }
-  static constexpr double kDrivetrainEncoderCountsPerRevolution() {
-    return 2048.0;
+  static constexpr double kRotationModuleRatio() {
+    return 9.0 / 24.0 * 14.0 / 72.0;
+  }
+
+  static constexpr double kTranslationModuleRatio() {
+    return 1.0 / 6.75 * 2.0 * .0254;
   }
 
   static constexpr double kMaxDrivetrainEncoderPulsesPerSecond() {

y2024_swerve/wpilib_interface.cc

diff --git a/y2024_swerve/wpilib_interface.cc b/y2024_swerve/wpilib_interface.cc
index 445a618..83d431c 100644
--- a/y2024_swerve/wpilib_interface.cc
+++ b/y2024_swerve/wpilib_interface.cc
@@ -257,10 +257,18 @@
             falcon->RefreshNontimesyncedSignals();
           }
 
-          modules.front_left->PopulateCanPosition(builder->add_front_left());
-          modules.front_right->PopulateCanPosition(builder->add_front_right());
-          modules.back_left->PopulateCanPosition(builder->add_back_left());
-          modules.back_right->PopulateCanPosition(builder->add_back_right());
+          const frc971::wpilib::swerve::SwerveModule::ModuleGearRatios
+              gear_ratios{
+                  .rotation = constants::Values::kRotationModuleRatio(),
+                  .translation = constants::Values::kTranslationModuleRatio()};
+          modules.front_left->PopulateCanPosition(builder->add_front_left(),
+                                                  gear_ratios);
+          modules.front_right->PopulateCanPosition(builder->add_front_right(),
+                                                   gear_ratios);
+          modules.back_left->PopulateCanPosition(builder->add_back_left(),
+                                                 gear_ratios);
+          modules.back_right->PopulateCanPosition(builder->add_back_right(),
+                                                  gear_ratios);
 
           builder.CheckOk(builder.Send());
         });