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

Filter setting sliders too aggressive (1.18.12 pre) #463

Closed
barjac opened this issue Jul 9, 2023 · 13 comments · Fixed by #485
Closed

Filter setting sliders too aggressive (1.18.12 pre) #463

barjac opened this issue Jul 9, 2023 · 13 comments · Fixed by #485

Comments

@barjac
Copy link

barjac commented Jul 9, 2023

Recently the level adjustments for the filters have become too course and difficult to set.
Reported to me by GW3OAJ on Windows and confirmed by me in Linux.

@tmiw
Copy link
Collaborator

tmiw commented Jul 9, 2023

How granular should the filter adjustments be? On macOS at least, each click of the slider seems to produce around a 0.4 dB adjustment right now.

@Tyrbiter
Copy link

Tyrbiter commented Jul 9, 2023

One thing I noticed is that the freq adjustments make it difficult to return to the original setting. So, for instance, 100Hz is always 96 or 102. Then 3000Hz is always 2964 or 3124.

One small thing, Miscellaneous is mis-spelt as "Miscallenous" in the USER_MANUAL.md and .html files.

@tmiw
Copy link
Collaborator

tmiw commented Jul 9, 2023

One thing I noticed is that the freq adjustments make it difficult to return to the original setting. So, for instance, 100Hz is always 96 or 102. Then 3000Hz is always 2964 or 3124.

Based on the current code, it seems that it assumes there are 100 possible positions for the sliders, which may not 100% be correct. I'm thinking for frequency sliders, there should be 10000 maximum positions, while anything involving magnitudes should be set somewhere that ensures 0.1 dB granularity. Does that sound reasonable or do you guys have any suggestions?

One small thing, Miscellaneous is mis-spelt as "Miscallenous" in the USER_MANUAL.md and .html files.

Yep, fixed in https://github.com/drowe67/freedv-gui/pull/466/files#diff-47b5dd4f4aceb13d5c25c92f0a31db61a9cb09e2a96eb3ce45be1db2dccbe9b2.

@Tyrbiter
Copy link

I'm not sure about the 10,000 positions, I suppose it's fine as long as there isn't a huge array to hold it.

0.1dB granularity might be a bit too granular, I wonder if 0.5dB would work or perhaps 0.25dB?

Other opinions welcomed :)

@tmiw
Copy link
Collaborator

tmiw commented Jul 11, 2023

Another possibility may be simply that the height of the sliders is too short to be able to properly adjust the settings, no matter what SLIDER_MAX is set to. I'm reluctant to try to make the window bigger as there have been reports before of various parts of the FreeDV UI not working well on smaller displays, at least so close to 1.8.12.

Anyway, we can play around more and figure out a solution that'll work. 👍

@Tyrbiter
Copy link

Perhaps the slider is the wrong thing for some settings, maybe a spin-wheel or similar would be better?

I don't see it as a very high priority item but it would be nice to improve it in the next release after 1.8.12 for those that use these settings (I have generally left them alone except for testing basic function).

@tmiw
Copy link
Collaborator

tmiw commented Jul 13, 2023

So, I actually wonder if this is the same issue mentioned on the digitalvoice list earlier today. Try v1.8.12 (currently building now) and let me know how that goes. 👍

@Tyrbiter
Copy link

So, I actually wonder if this is the same issue mentioned on the digitalvoice list earlier today. Try v1.8.12 (currently building now) and let me know how that goes. +1

I see you found some possible errors in the filter settings code. Makes sense to revisit now that the release is done. Well done @tmiw you have been burning the candle at both ends again ;-)

@barjac
Copy link
Author

barjac commented Jul 13, 2023 via email

@tmiw
Copy link
Collaborator

tmiw commented Jul 13, 2023

Yeah, the Default button sets the gain/Q to 0 across the board and the bass/mid/treble frequencies as 100, 1500 and 3000 Hz respectively. Apparently this didn't match up with the config file defaults either as if you wiped the settings, the frequencies were 1 Hz across the board.

Anyway, the bug was basically that everything was getting multiplied by 10 when you closed the Filter dialog. As a result, setting Q to e.g. 1 would cause it to get set to 10 when you reopened the dialog, which definitely isn't ideal.

@Tyrbiter
Copy link

I notice that the .freedv file has:

BassFreqHz=100
TrebleFreqHz=3000
MidFreqHz=1500

in it twice.

Is that intentional or does it need to have separate names?

@tmiw
Copy link
Collaborator

tmiw commented Jul 14, 2023

I notice that the .freedv file has:

BassFreqHz=100 TrebleFreqHz=3000 MidFreqHz=1500

in it twice.

Is that intentional or does it need to have separate names?

Intentional. There's one set with "MicIn" above it and another with "SpkOut".

@Tyrbiter
Copy link

I notice that the .freedv file has:
BassFreqHz=100 TrebleFreqHz=3000 MidFreqHz=1500
in it twice.
Is that intentional or does it need to have separate names?

Intentional. There's one set with "MicIn" above it and another with "SpkOut".

Oh yes, I should have grep'd above and below 'Freq'

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

Successfully merging a pull request may close this issue.

3 participants