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

Tweak ffmpeg Opus encoding optoins for more reliable audio #1299

Merged
merged 4 commits into from
Aug 20, 2023
Merged

Conversation

tsightler
Copy link
Collaborator

@tsightler tsightler commented Aug 20, 2023

These changes are pretty minor. Based on what I could determine, the audio from the doorbell was single channel, 24Khz sample rate, and a 24Kb bitrate, so this uses the same for the return audio. Also, disabled Opus variable bit rate, this seems to reduce the stuttering and overall robot sounding voice as well as works around the issues with recent werift versions.

The audio quality is still not as good as that from the Ring app, I'm not sure if this is due to Homekit audio, the transcoding itself (although I tried a lot of different options and this seemed to be the best), or if Ring is doing some type of audio filtering. The audio from Ring seems lower pitched and fuller, like they are filtering it through an equalizer or lowpass filter, which perhaps they are because the speakers in these devices don't do well with higher pitches.

I may play with this some, but for now, this seems better than the previous results, at least for the cameras I have access to.

@dgreif dgreif enabled auto-merge (rebase) August 20, 2023 18:32
@dgreif dgreif disabled auto-merge August 20, 2023 18:32
@dgreif dgreif enabled auto-merge (squash) August 20, 2023 18:33
@dgreif dgreif merged commit d6a5414 into main Aug 20, 2023
4 checks passed
@dgreif dgreif deleted the audio-tweaks branch August 20, 2023 18:33
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

Successfully merging this pull request may close these issues.

2 participants