Poor sound quality

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Use java-audio-player to play a high-quality MP3 (256kbps - like you would 
download from Amazon.com)

2. notice there is audible distortion

3. play the same MP3 directly with JLayer 1.0.1

4. notice there is no distortion

What is the expected output? What do you see instead?

I would like the sound quality to be just as good using java-audio-player as it 
is when I use JLayer 1.0.1 directly.

What version of the product are you using? On what operating system?

Source downloaded on December 11, 2011.  Running on Windows, through Eclipse.

Please provide any additional information below.

The specific track I'm testing with is "Rihanna - We Found Love"

Original issue reported on code.google.com by jonathan...@gmail.com on 12 Dec 2011 at 12:55

[GoogleCodeExporter]

I guess it could be caused by insufficient buffer size or output (to card) 
stream configuration.. probably I already have solution

Original comment by cy6erGn0m on 29 Dec 2011 at 12:47

• Changed state: Accepted
• Added labels: Priority-High
• Removed labels: Priority-Medium

[GoogleCodeExporter]

Original comment by cy6erGn0m on 29 Dec 2011 at 12:47

[GoogleCodeExporter]

I'm hoping to use this for New Year's Eve, so if there's any chance of solving 
the issue in the next 48 hours, I'd greatly appreciate it!

Thanks again for your hard work and assistance.

Original comment by jonathan...@gmail.com on 29 Dec 2011 at 1:08

[GoogleCodeExporter]

What is a frequency of your file? Can you see 44100 or 22050 in the terminal? I 
found issue with 22kHz is present on some windows too. So if you have 22050 
then you can just convert your file to mp3 with 44100 and everything should 
work.

Original comment by cy6erGn0m on 30 Dec 2011 at 6:46

[GoogleCodeExporter]

The sample rate is 44.1 kHz, and the bit rate is 256 kbps

Here's the file I'm using: 
http://dl.dropbox.com/u/3299396/Rihanna%20-%20We%20Found%20Love.mp3

Thanks again for your help

Original comment by jonathan...@gmail.com on 30 Dec 2011 at 11:41

[GoogleCodeExporter]

As I expected issue with this audio file caused by unsuitable buffer size 
autodetected by Java Sound Engine. Works fine with buffer size 88200 bytes. New 
build expected shortly..

Original comment by cy6erGn0m on 30 Dec 2011 at 3:44

• Changed state: Started

[GoogleCodeExporter]

Check for downloads: version 0.1.1.1 has been released. Please check whether it 
helps for you or not. For me it works fine under Java 7u1 and Linux with 
running pulseaudio and with Java 6u30 under Linux. Unfortunately I have no 
chance to test it under windows right now.

Original comment by cy6erGn0m on 30 Dec 2011 at 3:59

• Changed state: Fixed

[GoogleCodeExporter]

I suggest you to use 0.1.1.2 instead of 0.1.1.1

Original comment by cy6erGn0m on 30 Dec 2011 at 8:19

[GoogleCodeExporter]

This is terrific - you fixed it!

Thanks, cy6erGn0m!

(I noticed the source isn't updated yet - will you be pushing your changes?)

Original comment by jonathan...@gmail.com on 30 Dec 2011 at 10:12

[GoogleCodeExporter]

Everything in repo is up to date. You can see what was done to fix your issue 
and issue with 22kHz sound (that also can affect sound quality with 44kHz rate).

Original comment by cy6erGn0m on 31 Dec 2011 at 7:33

• Changed state: Verified

[GoogleCodeExporter]

See 
code.google.com/p/java-audio-player/source/detail?r=b13a5a4379e9745c226e0b78f030
12a11dbe37a2&repo=player
and
 code.google.com/p/java-audio-player/source/detail?r=8a4c8b17d46e39afe24f9cf2252f2bb7f4365503&repo=player

Original comment by cy6erGn0m on 31 Dec 2011 at 7:43

[GoogleCodeExporter]

Hey - just wanted to let you know that this was a HUGE hit at the New Years 
party.  Everyone was amazed by the software we were able to use based on this 
library.  I really appreciate your help in fixing the bug - thanks!

Original comment by jonathan...@gmail.com on 23 Jan 2012 at 3:09

[GoogleCodeExporter]

Thank you too for example of file that made possible to understand problem in 
library. 

Original comment by cy6erGn0m on 24 Jan 2012 at 2:02

• Changed state: Done