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

Mouse contronl on logarithmic models changes the value in linear scale #3871

Open
ghost opened this issue Oct 10, 2017 · 9 comments · May be fixed by #7647
Open

Mouse contronl on logarithmic models changes the value in linear scale #3871

ghost opened this issue Oct 10, 2017 · 9 comments · May be fixed by #7647
Labels

Comments

@ghost
Copy link

ghost commented Oct 10, 2017

Not all but some effects have much more value on one side then other making it hard to control.
Ill show example with : GLAME Butterworth Lowpass
Here is maximum :
image
Here is 50% :
image
And here is about 10% :
image
The difference from 100% to 50% is about 16733.
But from 50% to 10% its about 3000.
Tested this on version RC4.

@tresf
Copy link
Member

tresf commented Oct 10, 2017

@Texxiliator I think this is due to the plugin making incorrect use of the logarithmic flag.

port_range_hints[BWXOVER_IIR_CUTOFF].HintDescriptor =
LADSPA_HINT_BOUNDED_BELOW | LADSPA_HINT_BOUNDED_ABOVE | LADSPA_HINT_DEFAULT_LOW | LADSPA_HINT_SAMPLE_RATE | LADSPA_HINT_LOGARITHMIC;

See also #293.

@softrabbit
Copy link
Member

softrabbit commented Oct 11, 2017

Logarithmic is perfectly OK here. All these intervals are one octave and sound the same musically:

  • 100 - 200 Hz
  • 1000 - 2000 Hz
  • 10000 - 20000 Hz

That doesn't mean someone might need more precise control in the upper part of the range, but I believe that'd be less common.

@musikBear
Copy link

Imo the situation often with lmms dials, are that nothing or all comes 'out' within a very short span. The issue here, is that f.i. a filter pass can be quite difficult to automate. perhaps a 1.2.x could introduce something like an 'expander' in dials context menu, making the 'functional' span fit better inside the full span of the dial.

@PhysSong
Copy link
Member

Well, there are "Set linear"/"Set logarithmic" in context menus. If you don't like log scale, you may set that to linear.

perhaps a 1.2.x could introduce something like an 'expander' in dials context menu

<off-topic>
You might know, but 1.2 is on a feature freeze. New features are targeted to master branch.
If we do something known as "rolling release", however, you may use new features before 1.3 rc.
</off-topic>

@zonkmachine zonkmachine added the ux label Dec 14, 2017
@zonkmachine
Copy link
Member

Logarithmic response is what I'd expect to find on a filter cutoff control. In the example given has the correct control in my opinion.

PS. The resonance control on the filter above seem to be inverted.

@BaraMGB
Copy link
Contributor

BaraMGB commented Mar 8, 2018

Can this one to be closed? I can't see a bug here.

@DomClark
Copy link
Member

DomClark commented Mar 8, 2018

Not sure if this was what was meant originally, but whilst the scale on the dial is logarithmic, as makes sense for frequency, the mouse control isn't. Both the scroll wheel and dragging on the dial change the value by the same absolute amount no matter where the dial currently is, with the effect that it moves a lot faster for lower values as the logarithm increases more quickly there.

@Spekular
Copy link
Member

Spekular commented Mar 8, 2018 via email

@zonkmachine zonkmachine reopened this Mar 8, 2018
@zonkmachine
Copy link
Member

OK. We can look a bit closer at this. Reopening.

@husamalhomsi husamalhomsi removed the ux label Jul 31, 2019
@PhysSong PhysSong changed the title Effect Value Inequality Mouse contronl on logarithmic models changes the value in linear scale Mar 23, 2022
@PhysSong PhysSong added the gui label Mar 23, 2022
@regulus79 regulus79 linked a pull request Jan 12, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants