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

Track の stream_id が取れる SoraMediaTrack を追加する #34

Merged
merged 5 commits into from
Sep 5, 2023

Conversation

tnoho
Copy link
Contributor

@tnoho tnoho commented Aug 30, 2023

まさかの Stream ID を取得できない実装になっていたので修正します。

本来 Track には複数の Stream ID を紐づけることができるのですが、
Sora の使用上 Track には Stream ID が 1 つしか紐づかないため Track のメンバーとしました。

このクラスを継承している SoraAudioSource, SoraVideoSource については、

  • Connection にセットしてある際にセットされた組み合わせで Connection ごとに Stream ID が新規に設定されること
  • SoraAudioSource, SoraVideoSource は複数の Connection に割り当てが可能なこと

を踏まえて、この関数では std::nullopt (None) を返すこととしました。

SoraAudioSource, SoraVideoSource が SoraTrackInterface を継承しているのは、
将来的に OnTrack で受け取った SoraTrackInterface も Connection に AudioSource, VideoSource として設定できるようにしたいためです。

以上。レビューのほどよろしくお願いします。

@tnoho tnoho requested review from voluntas, sile and melpon August 30, 2023 15:05
@tnoho tnoho self-assigned this Aug 30, 2023
@tnoho
Copy link
Contributor Author

tnoho commented Aug 30, 2023

でも、もうこの段階で SoraConnection から on_track で渡すのを SoraTrackInterface を継承した SoraMediaTrack にして、そちらだけに stream_id を用意するのも手のような気もします。

src/sora_track_interface.h Outdated Show resolved Hide resolved
@voluntas
Copy link
Member

でも、もうこの段階で SoraConnection から on_track で渡すのを SoraTrackInterface を継承した SoraMediaTrack にして、そちらだけに stream_id を用意するのも手のような気もします。

👍 じゃぁそうしてみましょうか。PR 修正してください。

Copy link
Member

@voluntas voluntas left a comment

Choose a reason for hiding this comment

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

提案貰ったとおり SoraTrackInterface ではなく、それを継承した SoraMediaTrack を作りましょう。

@tnoho
Copy link
Contributor Author

tnoho commented Aug 31, 2023

SoraMediaTrack を作る作りに変更しました🙏

@voluntas
Copy link
Member

voluntas commented Sep 1, 2023

@tnoho タイトル変えた方がよさそう?

src/sora_track_interface.h Outdated Show resolved Hide resolved
@tnoho tnoho changed the title SoraTrackInterface に stream_id を追加する Track の stream_id が取れる SoraMediaTrack を追加する Sep 3, 2023
@voluntas
Copy link
Member

voluntas commented Sep 3, 2023

ドキュメント側がマージされたらこちらもマージします。

CHANGES.md Outdated Show resolved Hide resolved
@voluntas voluntas merged commit 6807c20 into develop Sep 5, 2023
15 checks passed
@voluntas voluntas deleted the feature/add-stream-id branch September 5, 2023 03:03
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