Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

Addressable LED Support #100

Open
Tropix126 opened this issue Mar 2, 2024 · 0 comments
Open

Addressable LED Support #100

Tropix126 opened this issue Mar 2, 2024 · 0 comments
Labels
scope:adi Updates ADI support

Comments

@Tropix126
Copy link
Member

Tropix126 commented Mar 2, 2024

Supersedes #59

What's the motivation for this feature?

PROS currently "supports" (more on this later) smart addressable LED control over ADI output. Unfortunately, this API is currently bugged in multiple ways (see: purduesigbots/pros#631).

Describe the solution you'd like

Implement non-broken addressible LED support. This will require:

Describe the drawbacks, if any

The daemon task implementation will not be pretty if we go that route.

Describe the alternative solutions, if any

Have LED setters block for 12ms to ensure they properly update for now, then fix it later on.

Additional context

None

@Tropix126 Tropix126 added the scope:adi Updates ADI support label Mar 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope:adi Updates ADI support
Projects
None yet
Development

No branches or pull requests

1 participant