| Summary: | JuK volume setting gets into infinite loop and uses incorrect rounding | ||
|---|---|---|---|
| Product: | [Applications] juk | Reporter: | David <david.cortes.rivera> |
| Component: | general | Assignee: | Scott Wheeler <wheeler> |
| Status: | REOPENED --- | ||
| Severity: | normal | CC: | kde, mpyne |
| Priority: | NOR | ||
| Version First Reported In: | 21.12.3 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
David
2021-10-05 03:27:25 UTC
A small correction about the change of volume: seems to come from calling method 'Phonon::AudioOutput::volume()' from 'PlayerManager::volume()', which returns something slightly different than what is set through 'PlayerManger::setVolume'. That is great that you have found the issue and have an idea for a fix. Whatever you can do is better than what we have now. I've gone through the bugs for juk that are related to "volume" and there are 4 others that seem to be pointing at the same issue that you are addressing, like https://bugs.kde.org/show_bug.cgi?id=452191 Mostly fixed in the latest version. Great! Are you referring to the upcoming 22.04 version of Juk? (In reply to Michael from comment #4) > Great! Are you referring to the upcoming 22.04 version of Juk? No, this is 21.12.3. I'm using 21.12.3. Are you saying that you haven't submitted your fix to the code base for next version? Because in this version, I'm not seeing the issue is fixed. (In reply to Michael from comment #6) > I'm using 21.12.3. Are you saying that you haven't submitted your fix to the > code base for next version? Because in this version, I'm not seeing the > issue is fixed. I have not submitted any fix. At least for me, it doesn't start muted now, but the volume bar does show it as being at -2^31 volume. I see. Well, would it be too much to ask for you to submit the code changes to the developer and leave it at that? The reason is that there are people like myself and one other person that I know personally for whom Juk starts out muted at every song change. It's unusable now. (In reply to Michael from comment #8) > I see. Well, would it be too much to ask for you to submit the code changes > to the developer and leave it at that? The reason is that there are people > like myself and one other person that I know personally for whom Juk starts > out muted at every song change. It's unusable now. I don’t think the code changes I tried would solve the issue and I don't even know how to submit a pull request, but if it’s of any use, I’ve since switched to Tauon Music box: https://tauonmusicbox.rocks Perhaps you should give it a try. I’ve found it to be a very suitable replacement if you tweak its config to mimic JuK, although you’ll need a few tricks to make it work correctly for KDE (e.g. set a kwin application rule so that it wouldn’t block compositing). If you know how to compile JuK (I did so by downloading the debian source package), you could also try hard-coding the volume to what you use in every volume variable referenced in the source code. Or perhaps updating QT would solve some of the issues you have. Hello, I haven't used Tauon Music box, but even if I did, it wouldn't help others to know about it and switch. It's not necessary to submit a pull request for simple code changes. In the past, I've just submitted an issue ticket with instructions for manual changes and that was enough. Also, I'd like to change the status of this bug to REOPENED as it's not truly fixed and your report is the most detailed filed so far. |