shiguredo / sora-ios-sdk

WebRTC SFU Sora iOS SDK
https://sora-ios-sdk.shiguredo.jp/
Apache License 2.0
42 stars 8 forks source link

映像解像度に 3840x2160 を追加する #163

Closed miosakuma closed 1 year ago

miosakuma commented 1 year ago

3840x2160 の解像度を増やす PR です。 ローカルネットワーク内ではこの解像度の送信に成功しています。

変数名は uhd2160p にしています。

気になる点 1

iPhone の設定画面では 4K という表記を使っていますが、数字先頭の変数名を使えないことや、今までの変数名と合わせた表記としてこの変数名にしています。

気になる点 2

4032x3024 の解像度でも送信はできたものの、iOS 独自のサイズであまり一般的な解像度と思えなかったため追加をしていません。 現在の最大解像度なので次世代にもっと一般的な解像度が出るのではないかと判断しました。 追加してもよいのではなどご意見あればお聞きしたいです。

szktty commented 1 year ago

@miosakuma

気になる点 1

uhd2160p でいいと思います。他の変数と揃ってますし不自然ではないと思います。

気になる点 2

動作に問題ないなら追加していいんじゃないでしょうか。検索してみると、 Android でも 4032x3024 対応のカメラを積んでいる端末はそこそこあるみたいです。

追加するとしたら変数名は uhd3024p で問題ないと思います。

miosakuma commented 1 year ago

コメントありがとうございました。 せっかくなので 4032x3024 も追加しました。 CHANGES.md はコンフリクトしてしまうのでマージ後に追加します。