Team2890/allwpilib
4
0
Fork 0
mirror of https://github.com/wpilibsuite/allwpilib synced 2026-06-24 01:31:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Increases tolerances for the PID tests"

Browse Source
This commit is contained in:
Brad Miller (WPI)
2014-10-22 11:36:36 -07:00
committed by Gerrit Code Review
parent 213e73c43e f7e8b95a24
commit 9e6d683671
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
wpilibj/wpilibJavaIntegrationTests/src/main/java/edu/wpi/first/wpilibj/PIDTest.java
View File

@@ -44,9 +44,8 @@ public class PIDTest extends AbstractComsSetup {
private static final Logger logger = Logger.getLogger(PIDTest.class.getName());
private NetworkTable table;
private static final double absoluteTollerance = 20;
private static final double outputRange = 0.19;
private static final double absoluteTollerance = 30;
private static final double outputRange = 0.2;
private PIDController controller = null;
private static MotorEncoderFixture me = null;
@@ -173,7 +172,7 @@ public class PIDTest extends AbstractComsSetup {
controller.disable();
controller.setSetpoint(setpoint);
controller.enable();
assertEquals(setpoint, new Double(controller.getSetpoint()));
assertEquals("Did not correctly set set-point",setpoint, new Double(controller.getSetpoint()));
}
@Test (timeout = 6000)
@@ -187,7 +186,7 @@ public class PIDTest extends AbstractComsSetup {
controller.enable();
Timer.delay(5);
controller.disable();
assertTrue(pidData() + "Controller Error: " + controller.getError(), controller.onTarget());
assertTrue(pidData() + "Was not on Target. Controller Error: " + controller.getError(), controller.onTarget());
}
private String pidData(){