Conversation
|
Pushed latest: CV Out Pitch/Gate/Velocity + LED feedback polish. |
Co-authored-by: Cursor <cursoragent@cursor.com>
7f0d229 to
c2afb8a
Compare
|
Written by an AI coding agent on @kosmar's behalf. Force-pushed a Wave-1 scope cleanup onto this branch (single commit vs
Please re-run CI on this branch. Happy to adjust anything for review. |
|
Hey @kosmar — two things, one about where this belongs and one about the manual format you're using. Where it belongs: this looks like a great fit for Ran About To resubmit there: just the app file itself (no |
|
Moving this to faderpunk-community-apps as requested: ATOVproject/faderpunk-community-apps#8 Closing here. App ID remapped 34 → 105 (community range). |
Summary
IDs 28–31 reserved for Heat Pump (#603), Grooves (#604), Golden Gate (#605), Super LFO (#606). Independent from
main.Test plan
Prerequisites
The platform pieces this app needs are under review as their own PRs, so that
each app PR stays small enough to review on its own.
Required — these APIs do not exist on current
main:App::clock_ticker, which replaces theClock::get_tickerthis branch was written againsttry_send_*)ParamStore::update, so an app can write its own params back to the hostThis branch is still based on an older
mainthat still had the removed clockhelper, which is why it builds on its own but not after a rebase.
One more dependency is deliberately not proposed yet:
App::wait_while_perf_muted, which this branch calls. That mechanism does notexist on
mainat all, and on the integration branch it is used by seven stockapps — so it needs its own PR and its own review rather than riding along here.
The call will be dropped from this branch until that PR exists.
Building any of this on a current Rust nightly also needs the toolchain fix in
#632.