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

[Bug]: The receiveGstreamerAudioVideo don't work in the v1.11.0 #2076

Open
3 tasks done
linzhenggang1 opened this issue Nov 6, 2024 · 4 comments
Open
3 tasks done
Labels
question Further information is requested v1.10.2 v1.11.0

Comments

@linzhenggang1
Copy link

Please confirm you have already done the following

  • I have searched the repository for related/existing bug reports
  • I have all the details the issue requires

Please answer the following prompt

  • This issue is replicable using the unmodified sample application

Describe the bug

When I ran the kvsWebrtcClientMasterGstSample, I found that it didn't play any audio.

I compared the source code of 1.10.1 and 1.11.0 and found that you deleted the code below in /samples/kvsWebrtcClientMasterGstSample.c from v1.11.0.

pSampleConfiguration->receiveAudioVideoSource = receiveGstreamerAudioVideo;

Is this a mistake?
If No, could you tell me how to make the kvsWebrtcClientMasterGstSample play audio from the remote peer?

Thank you.

Expected Behavior

The kvsWebrtcClientMasterGstSample plays audio from the remote peer.
(The receiveGstreamerAudioVideo task can run normally)

Current Behavior

The kvsWebrtcClientMasterGstSample doesn't play audio from the remote peer.
(The receiveGstreamerAudioVideo task doesn't run)

Reproduction Steps

  1. start the kvsWebrtcClientMasterGstSample as the Master
  2. start the https://awslabs.github.io/amazon-kinesis-video-streams-webrtc-sdk-js/examples/index.html as Viewer
  3. make some sound on the Viewer side and check whether the Master side play that

WebRTC C SDK version being used

v1.11.0

If it was working in a previous version, which one?

1.10.1

Compiler and Version used

gcc (Ubuntu 13.2.0-23ubuntu4) 13.2.0

Operating System and version

Ubuntu 24

Platform being used

Linux

@linzhenggang1 linzhenggang1 added bug Something isn't working needs-triage labels Nov 6, 2024
@sirknightj
Copy link
Contributor

Hi @linzhenggang1, thanks for your concern.
I was looking through the diff and wasn't able to find the line you mentioned above. Could you help me find it and/or the commit?
https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c/compare/v1.10.1..v1.11.0

To get the audio to playback for the MasterGstSample, you can add the following to https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c/blob/v1.11.0/samples/kvsWebrtcClientMasterGstSample.c#L392:

+    pSampleConfiguration->receiveAudioVideoSource = receiveGstreamerAudioVideo;
-    pSampleConfiguration->mediaType = SAMPLE_STREAMING_VIDEO_ONLY;
+    pSampleConfiguration->mediaType = SAMPLE_STREAMING_AUDIO_VIDEO;

@sirknightj sirknightj added question Further information is requested needs-more-information-from-requestor The requestor needs to provide more information/details to look into the issue. and removed bug Something isn't working needs-triage labels Nov 6, 2024
@linzhenggang1
Copy link
Author

@sirknightj
Thank you for your reply.

I found it from your URL link https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c/compare/v1.10.1..v1.11.0.
image

Maybe you have to click "Load diff" on the kvsWebrtcClientMasterGstSample.c section.

@sirknightj sirknightj added v1.10.2 v1.11.0 and removed needs-more-information-from-requestor The requestor needs to provide more information/details to look into the issue. labels Nov 7, 2024
@sirknightj
Copy link
Contributor

HI @linzhenggang1, thanks for the clarification.

I traced back the change to this pull request: #1972, which indicates it was removed intentionally due to issues encountered with the video track.

Are the changes I suggested above working for you to enable the audio playback you're looking for?

@linzhenggang1
Copy link
Author

linzhenggang1 commented Nov 7, 2024

Hi @sirknightj
Thanks for your work.

I tied the source code you suggested above in v1.11.0 and v1.10.2, but they didn't work and even didn't accept a CTRL + C command.

In addition, the v1.10.1 is working well in my environment.

Is the v1.11.0 and v1.10.2 well working in your environment?

When the receiveGstreamerAudioVideo thread does not work, the onGstAudioFrameReady function will never be called, so how will it play sounds?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested v1.10.2 v1.11.0
Projects
None yet
Development

No branches or pull requests

2 participants