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

Add GSMFullRateCompressor plugin. #72

Merged
merged 23 commits into from
Feb 8, 2022
Merged

Add GSMFullRateCompressor plugin. #72

merged 23 commits into from
Feb 8, 2022

Conversation

psobot
Copy link
Member

@psobot psobot commented Jan 30, 2022

This PR does what it says on the tin: adds a GSM, or "Full Rate" plugin, to make audio sound like it was transmitted over a 2G cell phone. To do this, this PR compiles in libgsm, a free, open-source, and ISC-licensed GSM encoder/decoder plugin.

@psobot psobot added the enhancement New feature or request label Jan 30, 2022
@psobot psobot changed the title Add GSM compressor. Add GSM compressor plugin. Jan 30, 2022
@psobot psobot force-pushed the psobot/gsm-compressor branch from 79733b5 to 417697f Compare January 31, 2022 01:59
Base automatically changed from psobot/mp3-compressor to master January 31, 2022 02:45
Copy link
Contributor

@drubinstein drubinstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm gonna be a little pedantic. I wouldn't call this a GSMCompressor but a GSMFullRateCompressor (though I'd prefer GSMFullRateEncoder) or allow this plugin to take in an enum which gives the appropriate codec parameters.. GSM supports multiple audio codecs including half rate, enhanced full rate and adaptive multi rate. HR and FR audio sounds incredibly different from each other.

@psobot
Copy link
Member Author

psobot commented Jan 31, 2022

Pedantry much appreciated here! I'll add FullRate somewhere in there. 👍🏻 Compressor vs Encoder is a tricky one, as neither is quite accurate (the audio is compressed/encoded, but then decompressed/decoded immediately after).

@psobot psobot changed the title Add GSM compressor plugin. Add GSMFullRateCompressor plugin. Jan 31, 2022
@drubinstein
Copy link
Contributor

drubinstein commented Jan 31, 2022

Then maybe not call it either and label the plugin GSMFullRate or GSMFullRateEffect

@psobot psobot force-pushed the psobot/gsm-compressor branch from 6d541f8 to f000972 Compare February 8, 2022 06:03
@psobot psobot merged commit 03eba78 into master Feb 8, 2022
@psobot psobot deleted the psobot/gsm-compressor branch February 8, 2022 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants