Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search




Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Welcome to the Rockbox Technical Forums!

+  Rockbox Technical Forums
|-+  Rockbox Development
| |-+  Feature Ideas
| | |-+  Smarter equalizer frequency scrolling in Advanced etc
« previous next »
  • Print
Pages: [1]

Author Topic: Smarter equalizer frequency scrolling in Advanced etc  (Read 2015 times)

Offline umptious

  • Member
  • *
  • Posts: 7
Smarter equalizer frequency scrolling in Advanced etc
« on: February 26, 2013, 06:27:06 PM »
At the moment it is linear, so getting from 10kHz to 12kHz takes a hell of a long time - even though the perceived difference between these two frequencies is quite small. This isn't too bad with a scroll wheel player, but with button driven players like the Clip it is unnecessarily awkward.

So I suggest that either

1. the increment should be geometrical

2. the increment should increase with the length of time that scrolling in the same direction continues

3. that increment switch from being 10s to 20s at say 2000Hz, go to 50s at 5000Hz, and then to 100s at 10kHz
Logged

Offline umptious

  • Member
  • *
  • Posts: 7
Re: Smarter equalizer frequency scrolling in Advanced etc
« Reply #1 on: February 26, 2013, 06:51:29 PM »
Ok.. the graphical equalizer is implemented in the same way. Same comment: this needs fixing! Hz are very misleading when you are thinking about human hearing: octaves are the true model of perception, and octavially speaking, the scroll rate reduces dramatically as you go up the frequency scale.
Logged

Offline saratoga

  • Developer
  • Member
  • *
  • Posts: 8767
Re: Smarter equalizer frequency scrolling in Advanced etc
« Reply #2 on: February 26, 2013, 07:30:43 PM »
This seems sensible enough, but I don't want to wade into the code badly enough to figure out how to change the increment to be nonlinear.
Logged

Offline JdGordon

  • Global Moderator
  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Smarter equalizer frequency scrolling in Advanced etc
« Reply #3 on: February 26, 2013, 11:42:54 PM »
Yeah, this isnt trivial to implement, but if I have some time I may have a look (If i remember).
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline umptious

  • Member
  • *
  • Posts: 7
Re: Smarter equalizer frequency scrolling in Advanced etc
« Reply #4 on: February 27, 2013, 09:22:38 AM »
I'm really surprised -  I thought the third method had to be simple with any imaginable code base. Naively, I suppose, I envisioned

freq += 10

being replaced by

if (freq < 500) freq += 10
else if (freq < 2000) freq += 20
else if (freq < 5000) freq += 50
else freq += 100

..just goes to show you!
Logged

Offline [Saint]

  • Rockbox Expert
  • Member
  • *
  • Posts: 1662
  • Hayden Pearce
    • Google+
Re: Smarter equalizer frequency scrolling in Advanced etc
« Reply #5 on: February 27, 2013, 06:01:58 PM »
I thought this was already possible actually...but, if it is, I can't seem to "make it go" :)

Perhaps it is, and is just non-functional for whatever reason. Last time I was sniffing around where the steps are defined for cutoff, q, freq etc (eq_menu.c/h?) I noticed defines for EQ_*_FAST_STEP which pairs with EQ_*_STEP which made me think that someone thought of this at some stage in the past. I didn't follow it far enough to check to see if it was just another case of some confusingly named defines.


[Saint]

EDIT: Note that I'm aware this isn't quite the same thing - it's not a nice ramped curve - it's simply two distinct values.
« Last Edit: February 27, 2013, 06:07:43 PM by [Saint] »
Logged
Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline JdGordon

  • Global Moderator
  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Smarter equalizer frequency scrolling in Advanced etc
« Reply #6 on: February 27, 2013, 10:46:48 PM »
I trhink the define you're talking about it used in the graphical eq screen, not the list (which should be doing acceleration automatically...)
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline [Saint]

  • Rockbox Expert
  • Member
  • *
  • Posts: 1662
  • Hayden Pearce
    • Google+
Re: Smarter equalizer frequency scrolling in Advanced etc
« Reply #7 on: February 28, 2013, 11:55:52 AM »
Quote from: Crafty on February 27, 2013, 11:16:41 PM
I think what you are seeing is somebody removed the .5 increcment but you still have to press the button TWICE!! to incroment 1db.

Nope...no sir, they did not (The relevant define there is EQ_GAIN_STEP, which you can see is set to 5, or, 0.5dB). IFF the Clip Zip indeed does behave as you describe, I should think it a bug. I haven't checked to confirm or deny the claim, but I can say that none of my other devices behave in the fashion you describe, the gain stepping is correctly presented in .5db increments - and I see no reason for this to be target-specific.


Quote from: JdGordon on February 27, 2013, 10:46:48 PM
I trhink the define you're talking about it used in the graphical eq screen, not the list (which should be doing acceleration automatically...)

It doesn't appear to be used there either...neither the GEQ or the lists appear to do so.


[Saint]

soap edit:  add opening bracket so url tag works.
« Last Edit: March 02, 2013, 09:53:40 AM by soap »
Logged
Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

  • Print
Pages: [1]
« previous next »
+  Rockbox Technical Forums
|-+  Rockbox Development
| |-+  Feature Ideas
| | |-+  Smarter equalizer frequency scrolling in Advanced etc
 

  • SMF 2.0.6 | SMF © 2013, Simple Machines
  • XHTML
  • RSS
  • WAP2

Page created in 0.09 seconds with 52 queries.