Merge branch 'release/2021.4.2'

CHANGES.md

@@ -11,6 +11,14 @@
 
 ## develop
 
+## 2021.4.2
+
+- [FIX] SetParameters() するタイミングを SetLocalDescription() の処理後に変更する事で Priority が動作するようにする
+    - @tsuyoshiii
+- [FIX] Priority から DegradationPreference への変換を実動作に合わせる
+- [UPDATE] cmake を 3.21.3 に上げる
+    - @voluntas
+
 ## 2021.4.1
 
 - [FIX] Windows 版リリース時の Invoke-WebRequest を curl に擬態する

VERSION

@@ -1,7 +1,7 @@
-MOMO_VERSION=2021.4.1
+MOMO_VERSION=2021.4.2
 WEBRTC_BUILD_VERSION=92.4515.9.2
 BOOST_VERSION=1.77.0
 CLI11_VERSION=2.0.0
 SDL2_VERSION=2.0.14
-CMAKE_VERSION=3.21.2
+CMAKE_VERSION=3.21.3
 CUDA_VERSION=11.0.2-1

doc/FAQ.md

@@ -94,6 +94,10 @@ NVENC が利用可能なビデオカードは以下で確認してください
 
 利用できません。60fps を使ってみたい場合は Jetson Nano などをご利用ください。
 
+## Sora モードで DataChannel メッセージングは利用できますか？
+
+Momo では Sora の DataChannel メッセージングに対応する予定はありません。 Sora C++ SDK にて対応予定です。
+
 ## Raspberry Pi (Raspberry-Pi-OS) で `--hw-mjpeg-decoder true` を指定した時に映像が出ません
 
 RaspberryPi の MJPEG デコーダ は一部の MJPEG に対応したカメラでしか機能しません。
@@ -106,4 +110,4 @@ Mac (arm64) から FHD でスクリーンキャプチャを配信したい場合
 
 設定方法はこちらの [Sora のドキュメント](https://sora-doc.shiguredo.jp/sora_conf#default-h264-profile-level-id)をお読みください。
 
-プロファイルレベル ID を変更しない場合は H.264 の HD 以下で配信するか、他のコーデックを使用して FHD 配信をしてください。
+プロファイルレベル ID を変更しない場合は H.264 の HD 以下で配信するか、他のコーデックを使用して FHD 配信をしてください。

src/ayame/ayame_client.cpp

@@ -236,6 +236,7 @@ void AyameClient::OnRead(boost::system::error_code ec,
     auto on_create_offer = [this](webrtc::SessionDescriptionInterface* desc) {
       std::string sdp;
       desc->ToString(&sdp);
+      manager_->SetParameters();
       boost::json::value json_message = {{"type", "offer"}, {"sdp", sdp}};
       ws_->WriteText(boost::json::serialize(json_message));
     };
@@ -262,6 +263,7 @@ void AyameClient::OnRead(boost::system::error_code ec,
               [this](webrtc::SessionDescriptionInterface* desc) {
                 std::string sdp;
                 desc->ToString(&sdp);
+                manager_->SetParameters();
                 boost::json::value json_message = {{"type", "answer"},
                                                    {"sdp", sdp}};
                 ws_->WriteText(boost::json::serialize(json_message));

src/momo_args.h

@@ -27,7 +27,7 @@ struct MomoArgs {
   std::string resolution = "VGA";
   int framerate = 30;
   bool fixed_resolution = false;
-  std::string priority = "BALANCE";
+  std::string priority = "FRAMERATE";
   bool use_sdl = false;
   bool show_me = false;
   int window_width = 640;

src/rtc/rtc_manager.cpp

@@ -236,13 +236,17 @@ void RTCManager::InitTracks(RTCConnection* conn) {
     webrtc::RTCErrorOr<rtc::scoped_refptr<webrtc::RtpSenderInterface>>
         video_add_result = connection->AddTrack(video_track_, {stream_id});
     if (video_add_result.ok()) {
-      rtc::scoped_refptr<webrtc::RtpSenderInterface> video_sender =
-          video_add_result.value();
-      webrtc::RtpParameters parameters = video_sender->GetParameters();
-      parameters.degradation_preference = config_.GetPriority();
-      video_sender->SetParameters(parameters);
+      video_sender_ = video_add_result.value();
     } else {
       RTC_LOG(LS_WARNING) << __FUNCTION__ << ": Cannot add video_track_";
     }
   }
 }
+
+void RTCManager::SetParameters() {
+  if (!video_sender_) { return; }
+
+  webrtc::RtpParameters parameters = video_sender_->GetParameters();
+  parameters.degradation_preference = config_.GetPriority();
+  video_sender_->SetParameters(parameters);
+}

src/rtc/rtc_manager.h

@@ -39,14 +39,13 @@ struct RTCManagerConfig {
   VideoCodecInfo::Type h264_encoder = VideoCodecInfo::Type::Default;
   VideoCodecInfo::Type h264_decoder = VideoCodecInfo::Type::Default;
 
-  std::string priority = "BALANCE";
+  std::string priority = "FRAMERATE";
 
-  // FRAMERATE が優先のときは RESOLUTION をデグレさせていく
   webrtc::DegradationPreference GetPriority() {
     if (priority == "FRAMERATE") {
-      return webrtc::DegradationPreference::MAINTAIN_RESOLUTION;
-    } else if (priority == "RESOLUTION") {
       return webrtc::DegradationPreference::MAINTAIN_FRAMERATE;
+    } else if (priority == "RESOLUTION") {
+      return webrtc::DegradationPreference::MAINTAIN_RESOLUTION;
     }
     return webrtc::DegradationPreference::BALANCED;
   }
@@ -63,11 +62,13 @@ class RTCManager {
       webrtc::PeerConnectionInterface::RTCConfiguration rtc_config,
       RTCMessageSender* sender);
   void InitTracks(RTCConnection* conn);
+  void SetParameters();
 
  private:
   rtc::scoped_refptr<webrtc::PeerConnectionFactoryInterface> factory_;
   rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track_;
   rtc::scoped_refptr<webrtc::VideoTrackInterface> video_track_;
+  rtc::scoped_refptr<webrtc::RtpSenderInterface> video_sender_;
   std::unique_ptr<rtc::Thread> network_thread_;
   std::unique_ptr<rtc::Thread> worker_thread_;
   std::unique_ptr<rtc::Thread> signaling_thread_;

src/sora/sora_client.cpp

@@ -409,7 +409,7 @@ void SoraClient::OnRead(boost::system::error_code ec,
             [self](webrtc::SessionDescriptionInterface* desc) {
               std::string sdp;
               desc->ToString(&sdp);
-
+              self->manager_->SetParameters();
               boost::asio::post(self->ioc_, [self, sdp]() {
                 if (!self->connection_) {
                   return;
@@ -443,6 +443,7 @@ void SoraClient::OnRead(boost::system::error_code ec,
             [self, answer_type](webrtc::SessionDescriptionInterface* desc) {
               std::string sdp;
               desc->ToString(&sdp);
+              self->manager_->SetParameters();
               boost::asio::post(self->ioc_, [self, sdp, answer_type]() {
                 if (!self->connection_) {
                   return;

src/util.cpp

@@ -149,7 +149,7 @@ void Util::ParseArgs(int argc,
   app.add_flag("--fixed-resolution", args.fixed_resolution,
                "Maintain video resolution in degradation");
   app.add_option("--priority", args.priority,
-                 "Preference in video degradation (experimental)")
+                 "Specifies the quality that is maintained against video degradation")
       ->check(CLI::IsMember({"BALANCE", "FRAMERATE", "RESOLUTION"}));
   app.add_flag("--use-sdl", args.use_sdl,
                "Show video using SDL (if SDL is available)")