- Status Unconfirmed
- Percent Complete
- 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
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.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
patch: malformed patch at line 105: @@ -414,9 +504,32 @@
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.]
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.]
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!
the patch seems have some problems, i tried to fix the patch error, but after compile, the EQ interface was dead…(cowon d2+)
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.
dreamlayers: have you checked actual equations used? Original author states that equations are wrong in original patch but he seems to abandon the idea.
No, I did not check the equations.