Equalizer for 0.3

Bug #304846 reported by Michal Dziczkowski
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
Medium
Unassigned

Bug Description

Exaile 0.3 needs an equalizer.

reacocard (reacocard)
Changed in exaile:
importance: Undecided → Medium
milestone: none → 0.3.0
status: New → Confirmed
Adam Olsen (arolsen)
description: updated
summary: - Not working equalizer
+ Equalizer for 0.3
Revision history for this message
reacocard (reacocard) wrote :

assinging to G since he said hed take it in irc yesterday.

Changed in exaile:
assignee: nobody → Abhishek Mukherjee (linkinpark342)
Revision history for this message
reacocard (reacocard) wrote :

deferring to 0.3.1 due to impending release.

Changed in exaile:
assignee: Abhishek Mukherjee (linkinpark342) → nobody
milestone: 0.3.0 → 0.3.1
Revision history for this message
Dave Aitken (dave-aitken) wrote :

Just finished writing one as a plugin, code is in this branch https://code.launchpad.net/~dave-aitken/exaile/exaile-eq

Revision history for this message
Nikola Kotur (kotnik) wrote :

Dave, it doesn't work. I just tested equalizer plugin against Exaile 0.3.0.1, and I get this error:

Traceback (most recent call last):
  File "/usr/share/exaile/plugins/equalizer/__init__.py", line 51, in _enable
    EQ_MAIN = EqualizerPlugin(exaile)
  File "/usr/share/exaile/plugins/equalizer/__init__.py", line 118, in __init__
    exaile.gui.builder.get_object('tools_menu').append(self.MENU_ITEM)
AttributeError: 'Main' object has no attribute 'builder'

Revision history for this message
Nikola Kotur (kotnik) wrote :

Dave, line 118 in eqaulizer/__init__.py should be changed from:

exaile.gui.builder.get_object('tools_menu').append(self.MENU_ITEM)

to:

exaile.gui.xml.get_widget('tools_menu').append(self.MENU_ITEM)

And than plugin can be successfully loaded in Exaile 0.3.0.1. But, nevertheless, the plugin is not doing anything to the sound, regardless of the settings.

Is this just a skeleton for the plugin?

Revision history for this message
Dave Aitken (dave-aitken) wrote :

Nikola - sorry for the slow reply. Not sure why that gui line change was needed, seemed to work before when I tested it. Unfortunately the equalizer won't work with the default sound engine due to a bug in that, but if you change to the unified (experimental) engine it should function properly. Although the unified sound engine also has some problems of it its own with playback.... so it's all a bit buggy at the minute unfortunately!

Revision history for this message
Nikola Kotur (kotnik) wrote :

Dave, thanks for the update.

I'll wait for the equalizer to land in 0.3.1 and than you'll have one happy user more. Unfortunately, I'm not that good with sound, otherwise I'd write it myself. But if I can help in any other way (testing, writing other code), please do not hesitate to ask.

Revision history for this message
reacocard (reacocard) wrote :

this was merged to trunk a while back in r2515. normal engine has a slight bug preventing it working, but it works with unified and fixing normal should be easy enough.

Changed in exaile:
status: Confirmed → Fix Committed
Revision history for this message
Nikola Kotur (kotnik) wrote :

Thank you Aren.

Testing it (basically, playing music) the whole day now, and it works just as expected.

reacocard (reacocard)
Changed in exaile:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.