Add Python control-side bindings (pyarco) - #5
Open
oltyan wants to merge 4 commits into
Open
Conversation
Pure-Python control bindings for Arco using o2litepy. Replaces direct O2 message construction with a Ugen class hierarchy that mirrors the Serpent bindings. - arco_engine.py: ArcoEngine lifecycle (connect/close/context manager), UgenID pool, action system, constants, utilities - arco_ugens.py: Ugen base class and ~50 concrete wrappers - arco_instr.py: Instrument framework (Param, Synth, Note/Score, etc.) - arco.py: re-export layer for backward compatibility - apps/test/python/init.py: NiceGUI interactive demo app ArcoEngine holds strong references to all ugen shadows, preventing accidental GC from sending premature /arco/free messages. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Brings the vendored pyarco/ copy up to date with the standalone Musical-Mycology/pyarco repo's memory-lifecycle overhaul: - weak-reference ugen registry (WeakValueDictionary) so dropping the last Python reference frees the server ugen and reclaims its pool slot - owns_id ownership rule (Instruments borrow their output ugen's id; no more per-Instrument slot leak / double-free) - client-side graph mirroring (container members, play/mute/fade pinning, borrow/set_alternate pinning) so nothing is GC'd while wired server-side - weak action targets with prune-on-delivery and per-callback isolation - Smoothb construction + newn payload fix; single-sourced ACTION_* consts - fade() race/double-fade guards; engine-scoped sawtooth singleton; per-thread instrument-construction stacks; Synth.is_finished releases the recycled note's Mix input - new offline pytest suite (pyarco/tests/, FakeO2Lite, 36 tests) — runs in the Arco tree with no server and no o2litepy dependency Vendored-location sys.path in arco_engine.py preserved (../apps/test/python); arco.py and init.py unchanged.
Brings the vendored copy up to Musical-Mycology/pyarco main: - arco_ugens.py: block-rate ugen constructors (Sineb, Resonb, Mathb, Unaryb, Tableoscb) now accept C_RATE Const inputs, rejecting only a-rate. Per Arco's own type system a `b` spec expands to `bc`. This unblocks Supersaw_instr construction, which was previously impossible. - tests: add test_rate_guards.py and test_supersaw.py (full Supersaw_synth noteon/noteoff cycle). Vendored suite is now 64 tests. - add pyarco/requirements-dev.txt (pins pytest for the offline suite). arco_engine.py / arco_instr.py / arco.py / init.py unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pyarco/— pure-Python control bindings for Arco using o2litepy, parallel to the Serpent bindings inserpent/srp/ArcoEnginemanages connection lifecycle and aUgenIDpool; live ugens are tracked in a weak registry so Python refcounting drives ugen lifetimeapps/test/python/init.pypyarco/tests/, 64 tests) that runs in the Arco tree with no server and no o2litepy dependencypyarco/README.mdfor architecture, the ugen lifecycle model, usage, and testing.Ugen lifecycle model
ArcoEngine._ugensis aweakref.WeakValueDictionary. Dropping the last Python reference to a pool-allocated ugen fires__del__, sends/arco/free, and returns the id to the pool — no session-long ID leak.inputsdict, containermembers(Sum/Sumb/Add/Addb/Route/Stdistr),Mix.inputs,play()/fade()output pinning, andborrow/set_alternateargument pinning.owns_id: pool-allocated ids are owned (freed by__del__); ids passed explicitly (system ugens, Instrument wrappers borrowing their output's id) are borrowed and never freed by the wrapper.atend/register_action) are held weakly, with dead-target pruning and per-callback exception isolation.Module layout
pyarco/README.mdpyarco/arco_engine.pypyarco/arco_ugens.pypyarco/arco_instr.pypyarco/arco.pyfrom arco import Sineworkspyarco/tests/pyarco/requirements-dev.txtapps/test/python/init.pyTest plan
python -m pytest pyarco/tests -v→ 64 passed (offline; no Arco server or o2litepy required), including a fullSupersaw_synthnoteon/noteoff cyclesource apps/common/setpath.sh && cd apps/test && ./daserpent.app/Contents/MacOS/daserpent)cd apps/test/python && python init.py)Slot is already free/No free slots)Known follow-ups
/actl/actis not yet registered as an o2lite handler and nothing callsatend(), so server-initiated note recycling (Synth.is_finished) doesn't fire against a live server yet (the method itself is correct and tested).term(dur)outside the fade helpers still desyncs the client pool from the server.🤖 Generated with Claude Code