-
Notifications
You must be signed in to change notification settings - Fork 2
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
v0.6.0 #59
base: main
Are you sure you want to change the base?
v0.6.0 #59
Conversation
FlowingSPDG
commented
May 20, 2024
•
edited
Loading
edited
- バックエンド
- vMixへの接続をキャッシュするように変更
- TCP接続へ変更
- フロントエンド
- バニラJSからReactへ変更
- 機能面
- パフォーマンスの改善
- Activators専用アクションの追加
TODO: 汚く書いてしまった部分をキレイにする |
あとログが散らばっているので1ファイルに固定したい |
React用に作った部分、割ときれいに作れたので型を整備してnpmに公開したい |
inputなど共通のコンポーネントを分割する |
いったん各種InputはNumberで指定する形に変更する |
TODO: マルチホストを追加した際にどこかでクラッシュが発生している。要調査。 恐らく |
vMixの再起動などで接続が変わったり、プロファイルが変わった際に設定が吹き飛ぶように見える |
InputNumber問題 ACTSコマンドが飛んできた際は、ほぼ何もせず(ActivatorNameのみ取得) 最悪の最悪、OnXMLにチャンネルを張って、OnActsの後XMLを送信/受信できるまではブロックしてもいいかもしれないが、何かのきっかけで即死しそうだし、複雑さがかなり上がる |
現時点でのビルド |
謎のpanicとTallyに反映されない問題(panic起因?)が起きている、要調査 |
#63 も手法としてある |
93146ca 時点でのビルド |
メモリ使用率・ディスク使用率の爆増がある |
TODO:
|
pool packageはcontroller, action 共に依存しているので、中に持たずにコールバックを外部登録にするか、interfaceを別packageに分ける |
1の方がコード綺麗になりそう Edit: 1にする |
TODO:
|
Inputsの受け渡しについて
|
双方向からのデータ受け渡し、vMixのシングルトン管理や責務分けを考えるときりがないので、多少汚くてもOKとする |