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

マルチコーデックサイマルキャストの送信に対応する #10

Open
wants to merge 36 commits into
base: develop
Choose a base branch
from

Conversation

melpon
Copy link
Contributor

@melpon melpon commented Feb 25, 2024

r0,r1,r2 に対応するコーデックを a=rid: の pt= に基づいて生成するようにしました。
pt= が無い場合、最初に見つかった a=rtpmap: のコーデックを利用します(既存の動作と同じ)。


This pull request includes several changes to the examples/sumomo files, focusing on improving the handling of optional parameters and updating the RTP (Real-time Transport Protocol) handling. It also includes updates to the CHANGES.md file and several changes to the proto/soracp.proto, include/sorac/, and src/ directories.

Updates to optional parameters handling:

Updates to RTP handling:

  • examples/sumomo/sumomo.c: Replaced rtp_encoding_parameters with rtp_parameters in the State struct and updated the on_capture_frame function to use rtp_parameters. [1] [2] [3]
  • examples/sumomo/util.cpp and examples/sumomo/util.h: Updated the sumomo_util_scale_simulcast function to take a scales array instead of a num_rids integer. [1] [2]
  • include/sorac/signaling.hpp: Updated the GetRtpParameters method in the Signaling class.
  • proto/soracp.proto: Updated the RtpEncodingParameters message and added the RtpParameters message. [1] [2]
  • src/signaling.cpp: Updated the GetRtpParameters method and the handling of RTP parameters in the SignalingImpl class. [1] [2] [3]

Other changes:

  • CHANGES.md: Added entries for supporting multi-codec simulcast transmission and updating protoc-gen-jsonif to support optional parameters.

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

Successfully merging this pull request may close these issues.

2 participants