Skip to content

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

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

feat(faderpunk): flash scene LED when clock is running#471
chmanie wants to merge 16 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.

ArthurGibert and others added 14 commits February 26, 2026 15:07
* fix(configurator): add markdown parser for app entries

* fix(configurator): add more chapter marks to manual
* fix(clock): add public tick counter, reset delay

* feat(clock): implement public tick counter in apps

* fix: use is_multiple_of where possible

---------

Co-authored-by: ArthurGibert <artgibert@gmail.com>
Bumps [keccak](https://github.com/RustCrypto/sponges) from 0.1.5 to 0.1.6.
- [Commits](RustCrypto/sponges@keccak-v0.1.5...keccak-v0.1.6)

---
updated-dependencies:
- dependency-name: keccak
  dependency-version: 0.1.6
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 6.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4...v6)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 5 to 7.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v5...v7)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/cache](https://github.com/actions/cache) from 4 to 5.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v4...v5)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 7.1.5 to 7.1.11.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v7.1.11/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-version: 7.1.11
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 5 to 6.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](actions/setup-node@v5...v6)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@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.

@chmanie chmanie force-pushed the main branch 4 times, most recently from d7ab681 to 18e6fd4 Compare March 16, 2026 19:10
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