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

Control of Voice Activity Detection (VAD) #129

Closed
aboba opened this issue Jul 14, 2014 · 2 comments
Closed

Control of Voice Activity Detection (VAD) #129

aboba opened this issue Jul 14, 2014 · 2 comments

Comments

@aboba
Copy link
Contributor

aboba commented Jul 14, 2014

In WebRTC 1.0 Section 4.2.4 we have:

dictionary RTCOfferOptions {
long offerToReceiveVideo;
long offerToReceiveAudio;
boolean voiceActivityDetection = true;
boolean iceRestart = false;
};

What is the equivalent means by which ORTC API controls voice activity detection?

@aboba
Copy link
Contributor Author

aboba commented Jul 15, 2014

In WebRTC 1.0 the voiceActivityDetection attribute determines what is put into the Offer (such as inclusion of CN as defined in RFC 3389). In ORTC API, we can control the codecs that are offered (such as CN) and/or the settings of those codecs (e.g. support for comfort noise in G.729). Therefore there is not a need for an explicit voiceActivityDetection attribute within RTCRtpParameters.

@aboba
Copy link
Contributor Author

aboba commented Jul 15, 2014

While no specification change is required, it would be useful to add a Section 16 on WebRTC compatibility issues, so as to answer this and other potential WebRTC compatibility questions. Here is the proposed text for a Section 16.1 within that section:

16.1 Voice Activity Detection

[WEBRTC10] Section 4.2.4 defines the RTCOfferOptions dictionary, which includes the voiceActivityDetection attribute, which determines whether Voice Activity Detection (VAD) is enabled within the Offer produced by createOffer(). The effect of setting voiceActivityDetection to TRUE is to include the Comfort Noice (CN) codec defined in [RFC3389] within the Offer.

Within ORTC API, equivalent behavior can be obtained by configuring the Comfort Noise codec for use within the RTCRtpParameters object, or configuring a codec with built-in support for Comfort Noise (such as G.729) to enable comfort noise.

robin-raymond pushed a commit to robin-raymond/ortc that referenced this issue Jul 16, 2014
…3c#66

Added Identity support, as described in Issue w3c#78
Reworked getStats method, as described in Issue w3c#85
Removed ICE restart method described in Issue w3c#93
Addressed CNAME and synchronization context issues described in Issue w3c#94
Fixed WebIDL issues noted in Issue w3c#97
Addressed NITs described in Issue w3c#99
DTLS transport issues fixed as described in Issue w3c#100
ICE transport issues fixed as described in Issue w3c#101
ICE transport controller fixes made as described in Issue w3c#102
Sender and Receiver object fixes made as described in Issue w3c#103
Fixed RTCRtpEncodingParameter default issues described in Issue w3c#104
Fixed 'Big Picture' issues descibed in Issue w3c#105
Fixed RTCRtpParameter default issues described in Issue w3c#106
Added a multi-stream capability, as noted in Issue w3c#108
Removed quality scalability capabilities and parameters, as described in Issue w3c#109
Added scalability examples as requested in Issue w3c#110
Addressed WebRTC 1.0 Data Channel compatibility issue described in Issue w3c#111
Removed header extensions from RTCRtpCodecParameters as described in Issue w3c#113
Addressed RTP/RTCP non-mux issues with IdP as described in Issue w3c#114
Added getParameter methods to RTCRtpSender and RTCRtpReceiver objects, as described in Issue w3c#116
Added layering diagrams as requested in Issue w3c#117
Added a typedef for payload type, as described in Issue w3c#118
Moved onerror from the RTCIceTransport object to the RTCIceListener object as described in Issue w3c#121
Added explanation of Voice Activity Detection (VAD), responding to Issue w3c#129
Clarified the meaning of maxTemporalLayers and maxSpatialLayers, as noted in Issue w3c#130
Added RFC 6051 to the list of header extensions and removed RFC 5450, as noted in Issue w3c#131
Addressed ICE terminology issues, as described in Issue w3c#132
Separated references into Normative and Informative, as noted in Issue w3c#133
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants