Skip to content
New issue

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

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

Already on GitHub? # to your account

add aec3 #580

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add aec3 #580

wants to merge 2 commits into from

Conversation

theomonnom
Copy link
Member

@theomonnom theomonnom commented Feb 20, 2025

https://github.com/webrtc-sdk/webrtc/blob/844bafa06d0b9088fd7fa4244832abf8e70a1d3d/modules/audio_processing/aec3/echo_canceller3.h#L120 isn't exposed, as it isn't needed for our local playout (no delay). This may need to be exposed for SDK like Unity

Copy link
Contributor

ilo-nanpa bot commented Feb 20, 2025

it seems like you haven't added any nanpa changeset files to this PR.

if this pull request includes changes to code, make sure to add a changeset, by writing a file to .nanpa/<unique-name>.kdl:

minor type="added" "Introduce frobnication algorithm"

refer to the manpage for more information.

Copy link
Member

@typester typester left a comment

Choose a reason for hiding this comment

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

LGTM.

This is exactly what I investigated earlier this month. Thank you for doing this.

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

Successfully merging this pull request may close these issues.

2 participants