Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Volume is boundless #150

Open
FossPrime opened this issue Aug 20, 2011 · 4 comments
Open

Volume is boundless #150

FossPrime opened this issue Aug 20, 2011 · 4 comments
Labels

Comments

@FossPrime
Copy link

Volume seems to go up to 11, causing distortion... well more like 10000%. The issue is that you never know if the volume is at 100%, specially when you listen to electronica.

I'll look into this and get back to you, I might just fix it my self.

@FossPrime
Copy link
Author

One line solution. On the other hand, volume down is also boundless.

app->player.scale = BarPlayerCalcScale (app->player.gain + ( app->settings.volume <= 0 ? app->settings.volume : --app->settings.volume));

@PromyLOPh
Copy link
Owner

I’ve no idea how to do this “correctly”. At the moment it’s just an ugly hack.

What do you define as “100%”, btw? settings.volume == 0?

@FossPrime
Copy link
Author

It's a hack if you consider the distortion a bug, you could call it a "digital boost support" feature for the libpiano. Clearly it is not a feature for the Pianobar front-end, if that's the case then that code is the right path.

0 is 100%, volume down seems to have some sort of logarithmic slope that never actually reaches 0%, although at 42 its pretty dead. So the question is, is it a bug, or a feature?

@PromyLOPh
Copy link
Owner

It's a hack if you consider the distortion a bug, you could call it a
"digital boost support" feature for the libpiano. Clearly it is not a
feature for the Pianobar front-end, if that's the case then that code
is the right path.
Uh, libpiano has nothing to do with it. See applyReplayGain in
src/player.c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants