Skip to content

Commit

Permalink
Remove useless SYSID stuff
Browse files Browse the repository at this point in the history
  • Loading branch information
GreenTomato5 committed Oct 7, 2024
1 parent 9c13af1 commit ab9a3b6
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 18 deletions.
2 changes: 1 addition & 1 deletion .github/scripts/constants.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
]

# Weird stuff that shouldn't go in constants, dont put function/var names in here theyre already checked
excused_cases = ["ModuleIOSparkMax", "case", "new Module(", "new BaseStatusSignal[", "BaseStatusSignal.waitForAll(", "new ModuleIOHybrid(", "Math.pow(", "+=", "drive.getRotation()", "autoChooser.addOption("]
excused_cases = ["ModuleIOSparkMax", "case", "new Module(", "new BaseStatusSignal[", "BaseStatusSignal.waitForAll(", "new ModuleIOHybrid(", "Math.pow(", "+=", "drive.getRotation()", "autoChooser.addOption(", "? -1 : 1"]

def check_for_magic_numbers(file_path):
magic_numbers = []
Expand Down
17 changes: 0 additions & 17 deletions src/main/java/frc/robot/subsystems/drive/Module.java
Original file line number Diff line number Diff line change
Expand Up @@ -247,18 +247,6 @@ public SwerveModuleState runSetpoint(SwerveModuleState state) {
return optimizedState;
}

/**
* Runs the module with the specified voltage while controlling to zero degrees.
*/
public void runCharacterization(double volts) {
// Closed loop turn control
angleSetpoint = new Rotation2d();

// Open loop drive control
io.setDriveVoltage(volts);
speedSetpoint = null;
}

/** Disables all outputs to motors. */
public void stop() {
io.setTurnVoltage(0.0);
Expand Down Expand Up @@ -313,9 +301,4 @@ public SwerveModulePosition[] getOdometryPositions() {
public double[] getOdometryTimestamps() {
return inputs.odometryTimestamps;
}

/** Returns the drive velocity in radians/sec. */
public double getCharacterizationVelocity() {
return inputs.driveVelocityRadPerSec;
}
}

0 comments on commit ab9a3b6

Please sign in to comment.