Rockbox

  • Status Unconfirmed
  • Percent Complete
    0%
  • Task Type Patches
  • Category User Interface
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.7.1
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Attilio Scotolati - 2011-01-06
Last edited by Alexander Levin - 2011-01-06

FS#11862 - Add a frequency response graph to the graphic equalizer menu

This patch adds a frequency response graph below the equalizer's controls, making it an EQ that is actually "graphical". :-)

I'm still not sure about the accuracy of the represented curve, in particular the Q factor on the peaking filters seems to have an effect opposite to the expected, though the equations are taken from the same "audio EQ cookbook" the equalizer is based on, any help is appreciated!

The user interface is still incomplete, I want to add proper tags on the axes and bars. I also thought about re-enabling the theme on this screen (why is it disabled?) and change the behavior of the "right" button, making it consistent with other menus.

Tested on a Sansa e200.

Marcin Bukat commented on 2011-01-08 13:19

patch: malformed patch at line 105: @@ -414,9 +504,32 @@

Hayden Pearce commented on 2011-01-08 17:19

I got the same here, was this patch built/edited by hand?

There are certainly some oddities present in the way the patch is laid out.
I managed to get it to compile, but I haven't tested it yet.

[St.]

Hayden Pearce commented on 2011-01-08 17:22

BTW, the EQ is already "graphical"…

What do you think you're seeing on screen when using the player other than…*graphics!* ;)
I think "Slightly more graphical than the already graphical graphical EQ" is a bit of a mouthful though.

[St.]

Attilio Scotolati commented on 2011-01-08 17:56

The EQ as of now is parametric with silders, which are a graphical element I admit, but IMHO not enough to make it graphical… Sorry for the patch not working, I've edited it by hand, I'm a newcomer to C and patches (had to learn FP programming too…), hence the mistake.
I've later found out that I was using the wrong equations to get the frequency response, so I'm now trying to address this, and meanwhile haven't a working patch. BTW I also have to think about smaller screens compatibility: free time is little and this could take a while!

JoshuaChang commented on 2011-01-14 00:37

the patch seems have some problems, i tried to fix the patch error, but after compile, the EQ interface was dead…(cowon d2+)

Boris Gjenero commented on 2011-11-15 17:29

I like the idea, as long as the graph shows the frequency response with reasonable accuracy. Here's a patch which applies cleanly on r30989 and works on the 5G iPod sim. Unfortunately, it doesn't work on my 5G iPod. I never actually saw the EQ screen there: one time I got a prefetch abort and another time I got a lockup.

BTW. recountdiff from patchutils helped with the original patch.

Marcin Bukat commented on 2011-11-15 17:44

dreamlayers: have you checked actual equations used? Original author states that equations are wrong in original patch but he seems to abandon the idea.

Boris Gjenero commented on 2011-11-18 03:37

No, I did not check the equations.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing