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

ビデオコーデック、オーディオコーデックのデフォルト値を未指定とする #115

Merged
merged 3 commits into from
Feb 20, 2025

Conversation

miosakuma
Copy link
Contributor

動作確認済みです

  • [CHANGE] VideoCodecType のデフォルト値を VP9 から未指定に変更する
    • 未指定の場合、シグナリング "type": "connect" でビデオコーデック指定を行わない
    • ビデオコーデック指定を行わない場合は Sora のデフォルト値 VP9 が利用されるため機能影響はない
  • [CHANGE] AudioCodecType のデフォルト値を OPUS から未指定に変更する
    • 未指定の場合、シグナリング "type": "connect" でオーディオコーデック指定を行わない
    • オーディオコーデック指定を行わない場合は Sora のデフォルト値 OPUS が利用されるため機能影響はない

This pull request includes changes to the Sora library to modify the default values for video and audio codec types. The most important changes include updating the default values for VideoCodecType and AudioCodecType to be unspecified, and adjusting the Connect method to handle these unspecified values properly.

Changes to default codec types:

  • Sora/Sora.cs: Changed the default value of VideoCodecType from VP9 to unspecified (VideoCodecType?).
  • Sora/Sora.cs: Changed the default value of AudioCodecType from OPUS to unspecified (AudioCodecType?).

Adjustments in the Connect method:

  • Sora/Sora.cs: Modified the Connect method to handle unspecified VideoCodecType by setting cc.video_codec_type to an empty string if config.VideoCodecType is null.
  • Sora/Sora.cs: Modified the Connect method to handle unspecified AudioCodecType by setting cc.audio_codec_type to an empty string if config.AudioCodecType is null.

Documentation updates:

  • CHANGES.md: Documented the changes to the default values for VideoCodecType and AudioCodecType.

@miosakuma miosakuma requested review from melpon and torikizi February 18, 2025 06:55
@miosakuma miosakuma changed the title ビデオコーデック、オーディオコーデックのデフォルト値を未指定とする [WIP] ビデオコーデック、オーディオコーデックのデフォルト値を未指定とする Feb 18, 2025
@miosakuma miosakuma force-pushed the feature/unset-default-codec branch from 55c77bf to 13f1670 Compare February 18, 2025 07:11
@miosakuma miosakuma changed the title [WIP] ビデオコーデック、オーディオコーデックのデフォルト値を未指定とする ビデオコーデック、オーディオコーデックのデフォルト値を未指定とする Feb 18, 2025
Co-authored-by: melpon <[email protected]>
Copy link
Contributor

@torikizi torikizi left a comment

Choose a reason for hiding this comment

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

修正ありがとうございます。
LGTM です。

Copy link
Contributor

@melpon melpon left a comment

Choose a reason for hiding this comment

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

よさそう

@miosakuma
Copy link
Contributor Author

確認ありがとうございます、マージします。

@miosakuma miosakuma merged commit 4b46c89 into develop Feb 20, 2025
@miosakuma miosakuma deleted the feature/unset-default-codec branch February 20, 2025 07:56
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.

3 participants