Skip to content

Commit

Permalink
Add a flag to disable vision simulation for performance reasons
Browse files Browse the repository at this point in the history
  • Loading branch information
charlesmackenzie committed Jan 27, 2024
1 parent ab4b6ad commit 187c579
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 4 deletions.
4 changes: 4 additions & 0 deletions src/main/java/frc/robot/Constants.java
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,10 @@ public static enum Mode {
REPLAY
}

public static final class FeatureFlags {
public static final boolean simulateVision = false;
}

public static final class ConversionConstants {
public static final double kRadiansPerSecondToRPM = 60.0 / (2.0 * Math.PI);
}
Expand Down
19 changes: 15 additions & 4 deletions src/main/java/frc/robot/RobotContainer.java
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import edu.wpi.first.wpilibj2.command.button.CommandJoystick;
import edu.wpi.first.wpilibj2.command.button.CommandXboxController;
import frc.robot.Constants.DriveConstants;
import frc.robot.Constants.FeatureFlags;
import frc.robot.Constants.TunerConstants;
import frc.robot.Constants.VisionConstants;
import frc.robot.commands.DriveWithJoysticks;
Expand All @@ -19,6 +20,7 @@
import frc.robot.subsystems.scoring.ScoringSubsystem;
import frc.robot.subsystems.scoring.ShooterIOSim;
import frc.robot.subsystems.scoring.ShooterIOTalon;
import java.util.Collections;

public class RobotContainer {
ScoringSubsystem scoringSubsystem;
Expand Down Expand Up @@ -111,10 +113,19 @@ public void configureSubsystems() {
new HoodIOSim(),
driveTelemetry::getFieldToRobot);

tagVision =
new VisionLocalizer(
new VisionIOSim(
VisionConstants.cameras, driveTelemetry::getModuleStates));
if (FeatureFlags.simulateVision) {
tagVision =
new VisionLocalizer(
new VisionIOSim(
VisionConstants.cameras,
driveTelemetry::getModuleStates));
} else {
tagVision =
new VisionLocalizer(
new VisionIOSim(
Collections.emptyList(),
driveTelemetry::getModuleStates));
}
break;
case REPLAY:
break;
Expand Down

0 comments on commit 187c579

Please sign in to comment.