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

Audio mix source #1597

Open
noho501 opened this issue Oct 8, 2024 · 8 comments
Open

Audio mix source #1597

noho501 opened this issue Oct 8, 2024 · 8 comments

Comments

@noho501
Copy link

noho501 commented Oct 8, 2024

Hi @pedroSG94 ,

Do you have any plans to improve the sound mixing quality when using both the microphone and in-app sound?
#1571

Currently, if both are used, the sound is not very clear. It seems like the microphone is picking up sound from both the speaker and microphone, making the live stream sound noisy and unpleasant.

If there is a way to mix the app sound and microphone audio more transparently, it would be great (I have already enabled echo cancellation and noise cancellation). I tried using the Larix app, and there is a noticeable difference in the sound quality.

Best regards,

@pedroSG94
Copy link
Owner

pedroSG94 commented Oct 8, 2024

Hello,

What do you means with noisy and unpleasant?
Do you have audio pops? echo?
Can you reproduce it in all devices?

@noho501
Copy link
Author

noho501 commented Oct 8, 2024

Yes, that's correct. I have tested both the RootEncoder and Larix apps, and below are the test links. The RootEncoder does not perform as well as Larix in terms of sound quality.

Larix: https://youtube.com/live/9tu5ujjUPxk
RootEncoder: https://youtube.com/live/ReBu0ijdaVc

@pedroSG94
Copy link
Owner

I see, I will check it.

For now, try add this line after create the stream class:

setTimestampMode(TimestampMode.BUFFER, TimestampMode.BUFFER)

This is only available in the master branch so you can use the app example with screen example and add that line here:
https://github.com/pedroSG94/RootEncoder/blob/master/app/src/main/java/com/pedro/streamer/screen/ScreenService.kt#L89

@pedroSG94
Copy link
Owner

I was doing tests using MixAudioSource and the audio sometimes produce noise but in general the quality is fine in my case. I tested using Pixel 6a and Xioami A3 (Android 14 and Android 10).

Also, after use TimestampMode.BUFFER, the quality is better and then I can't reproduce audio noise so this seem to solve the problem.

Let me know if you was able to fix the audio using that.

@noho501
Copy link
Author

noho501 commented Oct 9, 2024

Hi,

We will run a test and provide you with the results. Thank you for your help!

@noho501
Copy link
Author

noho501 commented Oct 12, 2024

Hi,

Thank you for your help. It seems to be working better than before.

@noho501 noho501 closed this as completed Oct 12, 2024
@noho501
Copy link
Author

noho501 commented Nov 9, 2024

Hi @pedroSG94 ,

We've encountered an issue with screen broadcasting that occurs after 20 minutes. At that point, an audio delay appears; for example, gunshots sound before they are visible in the video. You can see an example here: Delay.

Additionally, there's noticeable noise when the microphone is on, even though Noise Cancellation is enabled. Here’s an example: Noise.

Do you have any suggestions on how to address these issues?

@noho501 noho501 reopened this Nov 9, 2024
@pedroSG94
Copy link
Owner

Hello,

Ok, I will check it.
Thank you for the report

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

No branches or pull requests

2 participants