Skip to content

Commit b545f59

Browse files
Fix incorrect AddressableLED test end check and clarify Range record purpose
1 parent ea88276 commit b545f59

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/main/java/frc/robot/commands/test/AddressableLEDTestCommand.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public void execute() {
4848

4949
@Override
5050
public boolean isFinished() {
51-
return testCount < (AddressableLEDConstants.SECTIONS.length + 1);
51+
return testCount >= (AddressableLEDConstants.SECTIONS.length + 1);
5252
}
5353

5454
@Override

src/main/java/frc/robot/subsystems/addressableled/AddressableLEDConstants.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,9 @@
66

77
public class AddressableLEDConstants {
88
/**
9-
* @param low The lower bound of the range
10-
* @param high The upper bound of the range
9+
* This record is used to represent a range of LEDs that are grouped together and exhibit one pattern
10+
* @param low The lowest index LED in the range
11+
* @param high The highest index LED in the range
1112
*/
1213
public record Range(int low, int high) {}
1314

0 commit comments

Comments
 (0)