mirror of
https://github.com/devgianlu/go-librespot.git
synced 2026-04-26 05:15:49 +03:00
[PR #139] [CLOSED] fix: avoid popping noise when changing volume #229
Labels
No labels
bug
enhancement
pull-request
spotify-side
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/go-librespot#229
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/devgianlu/go-librespot/pull/139
Author: @aykevl
Created: 11/9/2024
Status: ❌ Closed
Base:
master← Head:alsa-volume-ramp📝 Commits (1)
7354f1efix: avoid popping noise when changing volume📊 Changes
1 file changed (+26 additions, -6 deletions)
View changed files
📝
output/driver-alsa.go(+26 -6)📄 Description
Just a little pet peeve of mine. When the volume changes (or is muted/unmuted), a little pop can be heard due to the sudden change. This can be fixed by smoothly increasing/decreasing the volume.
This noise is especially audible on tracks like this one: https://www.youtube.com/watch?v=eH-5U7GeofA
Weirdly, PulseAudio and PipeWire don't seem to do this internally. Maybe I'll send a patch to PipeWire to fix this, because it's a bit annoying (and there's not much we can do about it in the PulseAudio driver apart from managing volume ourselves).
So far this is an experiment, and there's a TODO I'd need to fix get this ready for review. But if this is something you'd be interested in, I can continue working on it.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.