Open otariidae opened 1 year ago
趣旨:websockets.broadcast 使えば良くね?
確かに a への代入は要らないと思います。 さらに言うとすべてのコルーチンの終了を待つには for 文ではなく次のように asyncio.gather の方が良いと思います。 asyncio.gather(*promises, return_exceptions=True) さらに言うと promises は次のように複数の接続に send する目的で使われていますが、それなら send ではなくwebsockets.broadcast の方が良い気がします。 https://github.com/ueckoken/plarail2022/blob/9afa0a0ad907a8276caf55b8134b22f4606e6b3c/frontend/videoCast/one_to_multiple_cast_skyway.py#L82-L95 こうなると変更範囲がこのPRのnitsにしては大きすぎると思うのでissueを立てて別PRでやることにします。
確かに a への代入は要らないと思います。
a
さらに言うとすべてのコルーチンの終了を待つには for 文ではなく次のように asyncio.gather の方が良いと思います。
for
asyncio.gather
asyncio.gather(*promises, return_exceptions=True)
さらに言うと promises は次のように複数の接続に send する目的で使われていますが、それなら send ではなくwebsockets.broadcast の方が良い気がします。
promises
send
websockets.broadcast
https://github.com/ueckoken/plarail2022/blob/9afa0a0ad907a8276caf55b8134b22f4606e6b3c/frontend/videoCast/one_to_multiple_cast_skyway.py#L82-L95
こうなると変更範囲がこのPRのnitsにしては大きすぎると思うのでissueを立てて別PRでやることにします。
_Originally posted by @otariidae in https://github.com/ueckoken/plarail2022/pull/140#discussion_r996441203_
@csenet WebRTCの配信方法を変更するからmulticasterは要らなくなる、みたいな話が確かあったかと思うのですが廃止することに決まったのでしょうか?
@otariidae WebSocketに関しては、1つのroomで実装するためmulticasterは必要なさそうです
趣旨:websockets.broadcast 使えば良くね?
_Originally posted by @otariidae in https://github.com/ueckoken/plarail2022/pull/140#discussion_r996441203_