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

Add opus sample frame generation GStreamer command to the readme #2098

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

sirknightj
Copy link
Contributor

Issue #, if available:

  • N/A

What was changed?

  • The readme has been updated with a sample GStreamer pipeline to generate some sample Opus frames.

Why was it changed?

  • The information did not exist in the readme and it is useful information for users to know to experiment.

How was it changed?

  • Added a sample command using audiotestsrc GStreamer element.
  • The new section was placed next to similar spot for the video frames generation.
  • Note that I was not able to locate the original music that it was generated from. It is likely the original author used filesrc and a local file instead of audiotestsrc.

What testing was done for the changes?

  • Locally replaced the sample frames and checked the playback with the JS page.

I also was able to get an mp3 audio to be played back on the JS page as well:

gst-launch-1.0 filesrc location="./My-Cool-Audio.mp3" ! decodebin ! audioconvert ! audioresample ! opusenc ! audio/x-opus,rate=48000,channels=2 ! multifilesink location="sample-%03d.opus" index=1

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Contributor

@stefankiesz stefankiesz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirmed these sample audio files stream e2e using Master sample and JS Viewer.

@sirknightj sirknightj merged commit 8b0bc13 into develop Jan 29, 2025
69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants