GNOME Bugzilla – Bug 619642
The default MP3 profile is broken
Last modified: 2010-09-28 01:48:03 UTC
Created attachment 161964 [details] [review] Change default to lamemp3enc encoding plugin By default the "CD Quality, MP3" profile is broken. It seems to encode to 128kbit CBR, while it intends to encode to VBR Quality 6. Changing the VBR Quality has no effect at all. I was told the 'lame' encoding plugin has been deprecated, and it's being replaced by 'lamemp3enc'. I attached a patch which changes the default, including the adding of a xing header. This will make sure Totem and others will properly display the file's bitrate and length. I hope this can be fixed for 2.30.1?
Small note, the attached patch is intended for a release tarball. So for upstream commit only the .in file needs to be modified I guess.
Yes, lame was deprecated by lamemp3enc and should always be used instead of lame (because of the broken interface and quality problems caused by this).
When was lamemp3enc added?
gst-plugins-ugly 0.10.12
Comment on attachment 161964 [details] [review] Change default to lamemp3enc encoding plugin Feel free to commit to 2.30.x and master
commit 155f6ce6463a95146664b08b924d057679ca2f7c Author: Pascal de Bruijn <pmjdebruijn@pcode.nl> Date: Sat May 29 07:54:58 2010 +0200 profiles: Use lamemp3enc and xingmux instead of the deprecated lame element The lame element has a broken interface and results in worse quality encodings. The xingmux element is added to allow correct duration reporting and seeking for VBR files. Fixes bug #619642. commit d81e10b63f5a38a5004ff2486e679608cb7dbab9 Author: Pascal de Bruijn <pmjdebruijn@pcode.nl> Date: Sat May 29 07:54:58 2010 +0200 profiles: Use lamemp3enc and xingmux instead of the deprecated lame element The lame element has a broken interface and results in worse quality encodings. The xingmux element is added to allow correct duration reporting and seeking for VBR files. Fixes bug #619642.
*** Bug 626286 has been marked as a duplicate of this bug. ***
I still have a small issue with the patch. quality=6 is far from CD quality; people with undamaged ears can easily hear the difference given a good enough listening environment. I would suggest setting quality=2 for the "CD quality" option. This produces significantly higher bitrates, so for users that are concerned about file size, you might want to introduce a new "portable MP3 player" option that sets quality=6. Lastly, here's the hydrogenadio suggested setting for "Voice quality": lamemp3enc name=enc target=1 bitrate=56 mono=true For more info, see http://wiki.hydrogenaudio.org/index.php?title=Lame#Recommended_encoder_settings https://bugs.launchpad.net/ubuntu/+source/sound-juicer/+bug/195483/comments/52
On second thought, this might go a bit beyond the scope of the current bug, so I filed it as a new one: https://bugzilla.gnome.org/show_bug.cgi?id=630779