- From: guidou via GitHub <sysbot+gh@w3.org>
- Date: Tue, 01 Apr 2025 10:13:10 +0000
- To: public-webrtc-logs@w3.org
guidou has just created a new issue for https://github.com/w3c/mediacapture-main: == Support multiple echoCancellation modes == The proposal here is allow multiple echo cancellation modes to better support certain use cases. Possible values: - `private`: The implementation tries to cancel every possible source of audio. This is useful, for example, to make sure audio from a screen reader or any other applications is removed from the microphone track. - `remote-only`: In this case only audio coming from remote peerconnections is cancelled. The use case here is something like music lessons, where the student plays an instrument together with some accompaniment played by the UA or some other application. In this case, the application wants to remove the audio from the teacher (which comes via a peerconnection), but not audio from the accompaniment. - `default` or `true`: The UA decides what to cancel. Equivalent to the current value of `true` for echoCancellation. - `off` or `false`: echo cancellation disabled. There are different ways to implement this. One way would be to change the property to be of type `(boolean or DOMString)`. It can take one of the above values (`true`, `false`, `"private"`, and `"remote-only"`). The advantage of this approach is that we reuse an existing property and existing applications will work without modification. A potential disadvantage is that a library that processes echoCancellation values will not be forward compatible with new applications that use the new values, but I don't think this is a common case. An alternative approach is to introduce a new echoCancellationMode constrainable property of type DOMString, in addition to the existing echoCancellation property. The advantage in this case is that everything using echoCancellation is backwards and forward compatible. The disadvantage is that we would have one property that is a superset of an existing property, which adds complexity and potential confusion. Eventually we'll probably want to deprecate and remove the old one. cc @alvestrand @jan-ivar @youennf Please view or discuss this issue at https://github.com/w3c/mediacapture-main/issues/1035 using your GitHub account -- Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Tuesday, 1 April 2025 10:13:11 UTC