Skip to content

feat(faderpunk): flash scene LED when clock is running#471

Open
chmanie wants to merge 5 commits intomainfrom
feat/clock-led-blink
Open

feat(faderpunk): flash scene LED when clock is running#471
chmanie wants to merge 5 commits intomainfrom
feat/clock-led-blink

Conversation

@chmanie
Copy link
Member

@chmanie chmanie commented Mar 4, 2026

Supersedes parts of #460. Flashes the scene button like a metronome when the clock is running.

@chmanie chmanie force-pushed the feat/clock-led-blink branch from 63f9ac7 to 3482224 Compare March 4, 2026 21:17
@chmanie chmanie changed the title feat(faderpunk): blink scene LED when clock is running feat(faderpunk): flash scene LED when clock is running Mar 4, 2026
@chmanie chmanie force-pushed the feat/clock-led-blink branch from 3482224 to 74ba1d8 Compare March 4, 2026 21:18
@ArthurGibert
Copy link
Contributor

Generally it works well but I think I preferred the way it looked before. A 1ppqn flash at High luminosity and the button stays at Mid the rest of the time.

Maybe we can have length and high and low luminosity as arguments to the effect to tune this

The idea of having the resolution as an argument was to be able to use the same effect in the apps to display resolutions. (which I ended up not doing so I don't think it's critical we go this way)

@ArthurGibert ArthurGibert force-pushed the feat/clock-led-blink branch from 8b7c556 to 46fa0b5 Compare March 6, 2026 09:37
Copy link
Contributor

@ArthurGibert ArthurGibert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice! I like it!

I just fixed a typo that was causing a compile issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants