refactor: Automatically assign column indexes.

.vscode/settings.json

@@ -25,5 +25,6 @@
       }
     },
   ],
-  "java.test.defaultConfig": "WPIlibUnitTests"
+  "java.test.defaultConfig": "WPIlibUnitTests",
+  "java.jdt.ls.vmargs": "-XX:+UseParallelGC -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90 -Dsun.zip.disableMemoryMapping=true -Xmx2G -Xms100m -Xlog:disable"
 }

src/main/java/frc/lib/Telemetry.java

@@ -1,11 +1,16 @@
 package frc.lib;
 
+import edu.wpi.first.wpilibj.shuffleboard.BuiltInLayouts;
 import edu.wpi.first.wpilibj.shuffleboard.Shuffleboard;
+import edu.wpi.first.wpilibj.shuffleboard.ShuffleboardLayout;
 import edu.wpi.first.wpilibj.shuffleboard.ShuffleboardTab;
+import java.util.HashMap;
 
 /** Helper class for managing robot telemetry. */
 public class Telemetry {
 
+  private static final HashMap<String, Integer> tabColumnCount = new HashMap<>();
+
   /**
    * Initializes a subsystem's Shuffleboard tab.
    *
@@ -18,4 +23,26 @@ public static void initializeShuffleboard(Subsystem subsystem) {
 
     subsystem.addToShuffleboard(tab);
   }
+
+  /**
+   * Adds a column to a Shuffleboard tab.
+   *
+   * @param tab the Shuffleboard tab to add the column to.
+   * @param columnTitle the title of the column.
+   * @return the added Shuffleboard column.
+   */
+  public static ShuffleboardLayout addColumn(ShuffleboardTab tab, String columnTitle) {
+    String tabTitle = tab.getTitle();
+
+    int currentColumnCount = tabColumnCount.getOrDefault(tabTitle, 0);
+    // Increment the column count for the next call
+    tabColumnCount.put(tabTitle, currentColumnCount + 1);
+
+    final int kColumnWidth = 2;
+    final int kColumnHeight = 4;
+
+    return tab.getLayout(columnTitle, BuiltInLayouts.kList)
+        .withSize(kColumnWidth, kColumnHeight)
+        .withPosition(currentColumnCount * kColumnWidth, 0);
+  }
 }

src/main/java/frc/robot/shooter/Shooter.java

@@ -1,11 +1,11 @@
 package frc.robot.shooter;
 
-import edu.wpi.first.wpilibj.shuffleboard.BuiltInLayouts;
 import edu.wpi.first.wpilibj.shuffleboard.ShuffleboardLayout;
 import edu.wpi.first.wpilibj.shuffleboard.ShuffleboardTab;
 import edu.wpi.first.wpilibj2.command.Command;
 import edu.wpi.first.wpilibj2.command.Commands;
 import frc.lib.Subsystem;
+import frc.lib.Telemetry;
 import frc.robot.shooter.BeamBreakSensorIO.BeamBreakSensorIOValues;
 import frc.robot.shooter.FlywheelMotorIO.FlywheelMotorIOValues;
 import frc.robot.shooter.SerializerMotorIO.SerializerMotorIOValues;
@@ -92,19 +92,16 @@ private double getFlywheelTangentialSpeed() {
 
   @Override
   public void addToShuffleboard(ShuffleboardTab tab) {
-    ShuffleboardLayout sensors =
-        tab.getLayout("Sensors", BuiltInLayouts.kList).withPosition(0, 0).withSize(2, 4);
+    ShuffleboardLayout sensors = Telemetry.addColumn(tab, "Sensors");
 
     sensors.addBoolean("Is Holding Note?", this::isHoldingNote);
 
-    ShuffleboardLayout serializer =
-        tab.getLayout("Serializer", BuiltInLayouts.kList).withPosition(2, 0).withSize(2, 4);
+    ShuffleboardLayout serializer = Telemetry.addColumn(tab, "Serializer");
 
     serializer.addDouble("Serializer Speed (mps)", this::getSerializerTangentialSpeed);
     serializer.addDouble("Serializer Current (A)", () -> serializerMotorValues.currentAmps);
 
-    ShuffleboardLayout flywheel =
-        tab.getLayout("Flywheel", BuiltInLayouts.kList).withPosition(4, 0).withSize(2, 4);
+    ShuffleboardLayout flywheel = Telemetry.addColumn(tab, "Flywheel");
 
     flywheel.addDouble("Flywheel Speed (mps)", this::getFlywheelTangentialSpeed);
     flywheel.addDouble("Flywheel Current (A)", () -> flywheelMotorValues.currentAmps);