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

VCSpeakerそのものが退出する順番とチャットの順番が違う #36

Open
book000 opened this issue Oct 20, 2023 · 7 comments

Comments

@book000
Copy link
Member

book000 commented Oct 20, 2023

本来「最後の人が退出する」→「VCSpeakerが退出する」なのに、チャットでは逆になっているように見えます。

Screenshot_20231020-180404

@yuuahp
Copy link
Member

yuuahp commented Oct 22, 2023


ここかも
私も前から気になってたけど何で直さなかったんだろ…

@book000
Copy link
Member Author

book000 commented Oct 23, 2023

guild.announce の下にくれば退出メッセージは入れ替わりそうねえ

@yuuahp
Copy link
Member

yuuahp commented Oct 23, 2023

ここかも
私も前から気になってたけど何で直さなかったんだろ…

思い出した
入退出処理を先に終わらせないとタイミングがずれるからだわ

@yuuahp
Copy link
Member

yuuahp commented Oct 23, 2023

ちょっと読み上げ処理見直すので時間ください

@book000
Copy link
Member Author

book000 commented Oct 23, 2023

タイミングがズレるってなんだ…まあこの問題は細かいところなので、大きな機能拡充とか安定化とかをしたほうが良いかも。

@yuuahp
Copy link
Member

yuuahp commented Oct 23, 2023

今、
VCSpeaker 参加・退出・移動 -> 読み上げ・送信 (ユーザー参加通知)
の順で処理してるので、
退出の時は変わらないけど、参加の場合、これを入れ替えると Narrator が生成されない (まだ VC に参加していない) まま読み上げ処理に移るから、結局読み上げられないことになる。
移動の場合は「yuua が ???? に参加しました」が「uua が ???? に参加しました」みたいな感じでずれる (多分 y が新しいチャンネルに移動する前に読まれてる)

@book000
Copy link
Member Author

book000 commented Oct 23, 2023

ああなるほどね…
数ミリ秒遅延させたらとか思ったけど、設計直しで解決するならその方がいいねえ

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

No branches or pull requests

2 participants