Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

LEDs #8

Merged
merged 11 commits into from
Jan 16, 2025
Merged

LEDs #8

merged 11 commits into from
Jan 16, 2025

Conversation

HenryLi-0
Copy link
Member

"Another really small project, and maybe doesn’t quite fall into fun stuff category, but you do the math:

For every year that I’ve been on this team, we have failed to have working LEDs
For every year that I’ve been on this team, we have had jank robots that have spent a good 30-50% of their time on the field disabled.
Correlation = causation
The only logical conclusion is that it is imperative for us to have working LEDs this year." -Siggy, Chief Delphi 2024

Uses the new WPILIB LEDs API!

@HenryLi-0 HenryLi-0 changed the title Led LEDs Jan 11, 2025
@HenryLi-0 HenryLi-0 marked this pull request as ready for review January 13, 2025 21:30
sigalrmp
sigalrmp previously approved these changes Jan 16, 2025
View reviewed changes
@sigalrmp sigalrmp dismissed their stale review January 16, 2025 02:35

oops! forgot about docs

@sigalrmp sigalrmp requested a review from kishan243 January 16, 2025 02:37
@HenryLi-0 HenryLi-0 merged commit 49b6efa into main Jan 16, 2025
2 checks passed
@Yxhej Yxhej deleted the led branch January 30, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kishan243 kishan243 kishan243 approved these changes

@sigalrmp sigalrmp sigalrmp approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@HenryLi-0 @sigalrmp @kishan243