Hook-based volume setter scripts run concurrently and potentially out-of-order
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 52/100
Research direction
Start by tracing volume handling in the daemon from the hook-based volume setter, then reproduce the issue by rapidly changing the volume with the slider. Check how concurrent hook calls are scheduled and ordered. Done means each requested volume, especially the final value, invokes the hook reliably and in the correct order.
Written by the indexing model from the issue text.
Description
It seems the hook-based volume setting command is sometimes not run.
In my setup, I run sendspin in a systemd unit via uvx:
uvx sendspin@7.5.0 daemon --audio-device=mydevice --hook-set-volume /usr/local/bin/set-alsa-volume.sh --name "myname" --id myid --disable-mpris --port 12345
My set-alsa-volume.sh script looks like this:
#!/bin/sh
exec systemd-cat -t "sendspin-triggered-amixer" amixer -D "$ALSA_DEVICE" sset "$ALSA_MIXER_CTL" "$1"%
(Note: The systemd-cat command ensures the output of the amixer command is sent to the journal of the sendspin systemd unit.)
Here's an example I managed to log where an expected call of the script does not occur:
Aug 04 14:36:10 server uvx[3235267]: INFO:sendspin.daemon.daemon:Server set player volume: 43%
Aug 04 14:36:10 server sendspin-triggered-amixer[294409]: Simple mixer control 'Master',0
Aug 04 14:36:10 server sendspin-triggered-amixer[294409]: Capabilities: volume cswitch
Aug 04 14:36:10 server sendspin-triggered-amixer[294409]: Playback channels: Front Left - Front Right
Aug 04 14:36:10 server sendspin-triggered-amixer[294409]: Capture channels: Front Left - Front Right
Aug 04 14:36:10 server sendspin-triggered-amixer[294409]: Limits: -50 - 100
Aug 04 14:36:10 server sendspin-triggered-amixer[294409]: Front Left: 14 [43%] [-25.80dB] Capture [on]
Aug 04 14:36:10 server sendspin-triggered-amixer[294409]: Front Right: 14 [43%] [-25.80dB] Capture [on]
Aug 04 14:36:11 server uvx[3235267]: INFO:sendspin.daemon.daemon:Server set player volume: 46%
Aug 04 14:36:11 server sendspin-triggered-amixer[294411]: Simple mixer control 'Master',0
Aug 04 14:36:11 server sendspin-triggered-amixer[294411]: Capabilities: volume cswitch
Aug 04 14:36:11 server sendspin-triggered-amixer[294411]: Playback channels: Front Left - Front Right
Aug 04 14:36:11 server sendspin-triggered-amixer[294411]: Capture channels: Front Left - Front Right
Aug 04 14:36:11 server sendspin-triggered-amixer[294411]: Limits: -50 - 100
Aug 04 14:36:11 server sendspin-triggered-amixer[294411]: Front Left: 19 [46%] [-24.30dB] Capture [on]
Aug 04 14:36:11 server sendspin-triggered-amixer[294411]: Front Right: 19 [46%] [-24.30dB] Capture [on]
Aug 04 14:36:14 server uvx[3235267]: INFO:sendspin.daemon.daemon:Server set player volume: 78%
Aug 04 14:36:14 server sendspin-triggered-amixer[294526]: Simple mixer control 'Master',0
Aug 04 14:36:14 server sendspin-triggered-amixer[294526]: Capabilities: volume cswitch
Aug 04 14:36:14 server sendspin-triggered-amixer[294526]: Playback channels: Front Left - Front Right
Aug 04 14:36:14 server sendspin-triggered-amixer[294526]: Capture channels: Front Left - Front Right
Aug 04 14:36:14 server sendspin-triggered-amixer[294526]: Limits: -50 - 100
Aug 04 14:36:14 server sendspin-triggered-amixer[294526]: Front Left: 67 [78%] [-9.90dB] Capture [on]
Aug 04 14:36:14 server sendspin-triggered-amixer[294526]: Front Right: 67 [78%] [-9.90dB] Capture [on]
Aug 04 14:36:16 server uvx[3235267]: INFO:sendspin.daemon.daemon:Server set player volume: 68%
<expected call of set-alsa-volume.sh does not occur here>
The example was produced by quickly changing the volume many times in the Music-Assistant web UI using the volume slider. The last setting (68%) does not result in a corresponding call of the volume setter hook.
Could the problem be a race condition between some internal state that represents the current volume and some check that decides whether to call the script?
EDIT: I've changed the title of the issue to better reflect the findings in my additional comment below.
- Dominant language
- Python
- Stars
- 190
- Forks
- 36
- Avg merge
- 11d 19h
- Merged PRs (30d)
- 2
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Sendspin/sendspin-python-cli
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Sendspin/sendspin-python-cli#280 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Sendspin/sendspin-python-cli#278 · 2 comments · 5 reactions ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Sendspin/sendspin-python-cli#257 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Sendspin/sendspin-python-cli#256 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Sendspin/sendspin-python-cli#113 · 10 comments ·
All issues in Sendspin/sendspin-python-cli
Similar issues
-
area/auth bug comp/agent P3 platform/discord type/security
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
NousResearch/hermes-agent#117848 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
bancolombia/sentinel#23 ·
-
test md OpenCI
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
integration:quickjs org:external priority:backlog topic:code-interpreter topic:middleware type:feature
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
langchain-ai/deepagents#6450 ·
-
bug client
Difficulty 2/5 1-3 hours Newbie friendliness 88/100