Skip to content

RTC: Fix FFmpeg opus audio noisy issue. v5.0.195 v6.0.95#3845

Merged
winlinvip merged 8 commits intoossrs:developfrom
xiaozhihong:bugfix/ffmpeg_opus
Oct 25, 2023
Merged

RTC: Fix FFmpeg opus audio noisy issue. v5.0.195 v6.0.95#3845
winlinvip merged 8 commits intoossrs:developfrom
xiaozhihong:bugfix/ffmpeg_opus

Conversation

@xiaozhihong
Copy link
Copy Markdown
Collaborator

@xiaozhihong xiaozhihong commented Oct 19, 2023

Follow the example in FFmpeg's doc, before calling the API avcodec_send_frame, always use av_frame_alloc to create a new frame.


Co-authored-by: Haibo Chen 495810242@qq.com

@winlinvip winlinvip added the EnglishNative This issue is conveyed exclusively in English. label Oct 19, 2023
@xiaozhihong
Copy link
Copy Markdown
Collaborator Author

Please apply the PR, remove obj dir, and use --ffmpeg-opus=on to configure, ensuring the audio play is not noisy.

@winlinvip winlinvip changed the title RTC: Fix FFmpeg opus audio noisy issue RTC: Fix FFmpeg opus audio noisy issue. v5.0.194 v6.0.94 Oct 21, 2023
@winlinvip winlinvip added the RefinedByAI Refined by AI/GPT. label Oct 21, 2023
@winlinvip winlinvip changed the title RTC: Fix FFmpeg opus audio noisy issue. v5.0.194 v6.0.94 RTC: Fix FFmpeg opus audio noisy issue. Oct 21, 2023
@winlinvip
Copy link
Copy Markdown
Member

winlinvip commented Oct 25, 2023

The default configuration for version 6.0 is --ffmpeg-opus=on, while for version 5.0, it's --ffmpeg-opus=off. Please handle this when merging.

TRANS_BY_GPT4

@winlinvip winlinvip changed the title RTC: Fix FFmpeg opus audio noisy issue. RTC: Fix FFmpeg opus audio noisy issue. v5.0.195 v6.0.95 Oct 25, 2023
@winlinvip winlinvip merged commit 9238f09 into ossrs:develop Oct 25, 2023
winlinvip pushed a commit that referenced this pull request Oct 25, 2023
Follow the example in FFmpeg's doc, before calling the API
`avcodec_send_frame`, always use `av_frame_alloc` to create a new frame.

---------

Co-authored-by: Haibo Chen <495810242@qq.com>
duiniuluantanqin added a commit to duiniuluantanqin/srs that referenced this pull request Oct 27, 2023
Follow the example in FFmpeg's doc, before calling the API
`avcodec_send_frame`, always use `av_frame_alloc` to create a new frame.

---------

Co-authored-by: Haibo Chen <495810242@qq.com>
chundonglinlin added a commit to chundonglinlin/srs that referenced this pull request Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

EnglishNative This issue is conveyed exclusively in English. RefinedByAI Refined by AI/GPT.

3 participants