Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

can't encode mp3s from 24 bit #36

Closed
custerflux opened this issue Jul 26, 2020 · 6 comments
Closed

can't encode mp3s from 24 bit #36

custerflux opened this issue Jul 26, 2020 · 6 comments

Comments

@custerflux
Copy link

Max version 0.9.2b2 doesn't seem to be able to process 24-bit audio into MP3 correctly. Whatever the samplerate, 24-bit audio gets encoded into total silence. It will encode properly to other formats besides MP3, and 16-bit audio seems to pass through everything just fine.

@timnyss
Copy link

timnyss commented Sep 12, 2020

Same issue. This only happened since I updated to Catalina a few weeks ago.

@sbooth sbooth closed this as completed in 4f66554 Sep 13, 2020
@sbooth
Copy link
Owner

sbooth commented Sep 13, 2020

This was an interesting bug that also affected to 32-bit source data. lame_encode_buffer_long2() expects the values to be scaled to fit the size of long, not a particular bit width. Max was assuming 32-bit scaling.

@sbooth
Copy link
Owner

sbooth commented Sep 14, 2020

Max-0.9.2b3 is now available and contains the fix.

@timnyss
Copy link

timnyss commented Sep 14, 2020 via email

@custerflux
Copy link
Author

Works great! Thank you.

@sbooth
Copy link
Owner

sbooth commented Sep 15, 2020

In hindsight it would be easier to use lame_encode_buffer_ieee_float() for all bit depths but I don't think that API existed in LAME 3.95 when this code was originally written.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants