Merge remote-tracking branch 'origin/arm-limit-switches' into arm-target-height

This commit is contained in:
Keenan D. Buckley
2019-03-09 13:15:09 -07:00
@@ -21,6 +21,7 @@ import com.ctre.phoenix.motorcontrol.NeutralMode;
import com.ctre.phoenix.motorcontrol.SensorCollection; import com.ctre.phoenix.motorcontrol.SensorCollection;
import com.ctre.phoenix.motorcontrol.can.WPI_TalonSRX; import com.ctre.phoenix.motorcontrol.can.WPI_TalonSRX;
import edu.wpi.first.wpilibj.DigitalInput;
import edu.wpi.first.wpilibj.Solenoid; import edu.wpi.first.wpilibj.Solenoid;
import edu.wpi.first.wpilibj.command.Subsystem; import edu.wpi.first.wpilibj.command.Subsystem;
import edu.wpi.first.wpilibj.smartdashboard.SmartDashboard; import edu.wpi.first.wpilibj.smartdashboard.SmartDashboard;
@@ -103,6 +104,9 @@ public class Arm extends Subsystem implements ControlLoopable
private PIDParams armPIDParams = new PIDParams(P_Value, I_Value, D_Value, KF_DOWN); // KF gets updated later private PIDParams armPIDParams = new PIDParams(P_Value, I_Value, D_Value, KF_DOWN); // KF gets updated later
public static final double PID_ERROR_INCHES = 5.0; public static final double PID_ERROR_INCHES = 5.0;
LimitSwitchSource limitSwitchSource; LimitSwitchSource limitSwitchSource;
DigitalInput forwardLimitSwitch = new DigitalInput(1);
DigitalInput reverseLimitSwitch = new DigitalInput(2);
// Pneumatics // Pneumatics
private Solenoid speedShift; private Solenoid speedShift;
@@ -296,6 +300,10 @@ public class Arm extends Subsystem implements ControlLoopable
} }
lastControlLoopUpdateTimestamp = currentTimestamp; lastControlLoopUpdateTimestamp = currentTimestamp;
if (reverseLimitSwitch.get()){
motor1.setPosition(0);
}
// Do the update // Do the update
if (armControlMode == ArmControlMode.JOYSTICK_MANUAL) { if (armControlMode == ArmControlMode.JOYSTICK_MANUAL) {
controlManualWithJoystick(); controlManualWithJoystick();