-
Notifications
You must be signed in to change notification settings - Fork 199
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
feat: Add AudioOptions class #978
base: main
Are you sure you want to change the base?
Conversation
So hey, I've followed the issues that are mentioned in this PR and I also would like to give a try to the audio options. Sadly I have problems with getting this specific pr to build as I'm not very well versed with CMake and the like. It throws an error about not finding presets and shuts down. Anyhow, is there any progress on making this a reality? Any sort of easier way I can add this and experiment with it in my own project yet? |
@flamacore Here you are. Please install the tgz file following this manual. |
@karasusan would be delighted to. Thanks a lot for sharing the package. I'll give it a check soon and will inform if it works or not. |
I tested and issue #964 remains. #...
{
#...
var track = new AudioStreamTrack(sendAudioSource, GetAudioOptions());
mediaStream.AddTrack(track);
}
private AudioOptions GetAudioOptions()
{
return new AudioOptions()
{
echoCancellation = true,
noiseSuppression = true
};
} |
Can confirm the echo issue persists. Not sure if it's related to us enabling/disabling the options or if the internal echo cancellation works as intended. EDIT: I tried it multiple ways:
|
Thank you for checking. I'd like to clarify which this issue is a specific by mobile platforms or not. It looks there are different implementations for mobile and desktop platforms. |
@karasusan |
@MaximKurbanov I can't share you many information about that, but at least I can say that the "audio processing" in native code is not processed. The "audio processing" in webrtc is implemented here. |
@karasusan This is a show-stopper for us. So we really, really need this fix in the next release (pre-8). 🙏
がんばってよ! |
We found a workaround on iOS involving |
This PR adds the AudioOptions class to set audio options to tracks.
The options are defined by the AudioOptions class.
This PR makes AudioStreamTrack constructor changing.
AudioStreamTrack(AudioOptions options = null);
AudioStreamTrack(AudioSource source, AudioOptions options = null)
AudioStreamTrack(AudioListener listener, AudioOptions options = null)