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

エンジンごとのデフォルト辞書も同期できるようにする #575

Closed
3 tasks
takana-v opened this issue Jan 14, 2023 · 6 comments
Closed
3 tasks

Comments

@takana-v
Copy link
Member

内容

現在の仕様では、それぞれのエンジンのデフォルト辞書にそれぞれのオリジナルキャラの読みが登録されています。
よって、エンジンを跨ぐとキャラの名前が読めない、ということが発生するかもしれません。
これを解決したいです。

Pros 良くなる点

エンジンを跨いでもキャラクターの名前を正しく読める

Cons 悪くなる点

実現方法

  • DefaultDictCSVLoaderみたいなクラスを実装
  • --additional_default_dict_csv引数を追加
  • エンジン起動時にLoaderを作成し、generate_appに渡す

VOICEVOXのバージョン

0.?.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

もう少し良い実装案が出るかもしれないので「要議論」をつけておきます。

@Hiroshiba
Copy link
Member

Issueありがとうございます!!間違いなく課題だなと思いました!

実現方法に書かれていることって、誰かが全エンジンのdefault csvを結合して、それを引数で指定する…みたいな感じでしょうか👀

@takana-v
Copy link
Member Author

自分の中では、全エンジンのdefault.csvのパスをエンジンに渡し、エンジン側でcsvを読み込んで結合するイメージでした。
--additional_default_dict_csvは複数取れるようにする)
ただこれはエディタ側で処理しても問題ないと思います。

また、エンジンのデフォルト辞書の場所はengine_manifestに書くのが良いかなと思います。

@Hiroshiba
Copy link
Member

Hiroshiba commented Jan 14, 2023

なるほどです!
エンジン側の実装は素直なのですが、エディタ側から見ると引数の指定と全エンジン再起動が必要になりそうです。
特に全エンジン再起動はUX的に謎の待機が発生するのでもったいないな~と感じました。

アイデアとしては、additional_default_dict_csvに該当するものをこの前できたsetting.yamlに保存するようにし、設定APIを介して制御するのも良いかもと思いました!
パスをAPIで指定するのは若干危険な気と、なんか設計的にイマイチな気もしますが。。。

あるいはユーザー辞書をimport/exportできるのと同じようにして、デフォルト辞書もimport/export可能にするのもありかもと思いました。
こっちのが仕様が揃うので、仮に将来変更するとしても簡単かもです。

@tarepan
Copy link
Contributor

tarepan commented Mar 6, 2024

本 issue に関して、必要性があると判断され設計は議論中との認識です。

@Hiroshiba
2024年3月現在でも上記のデフォルト辞書同期機能は有用そう、という認識でしょうか?

その場合、本 issue は最終議論から 1年以上経過しており、「有用だしどちらかといえば賛成なのですが、手が回らずかつすぐに手を回せる人もいないというタスク」のように見受けられます。
ロードマップ #1106 への登録と一旦 close が1つの方向性として有り得そうですが、どうでしょうか?

@tarepan tarepan self-assigned this Mar 7, 2024
@Hiroshiba
Copy link
Member

Hiroshiba commented Mar 16, 2024

@tarepan 取り上げありがとうございます!

マルチエンジン実装から1年が経って状況が結構見えてきたので重要度を再考した感じだと、

  • 設計の難度が割と高い(実装に時間がかかる)こと
  • 需要がそこまで大きくないだろうということ(これは時間経過でわかった)
  • 迂回策がかなり簡単なこと(エディターで辞書登録すればいい)

から、重要度は高くないかもと感じました!
ロードマップ #1106 に登録するか、需要が発生するまで実装を見送るかどちらかかなと思っていてちょっと迷ってます。

まーーーー需要の声を聞くまでクローズでも良いかも・・・?

@tarepan
Copy link
Contributor

tarepan commented Mar 17, 2024

需要の声を聞くまでクローズでも良いかも

👍
マルチエンジンがより普及すれば自然と需要が出てきそうなので、close しても必要に応じてこの issue が発掘される気がします。
ゆえに現段階での close に同意です。

@takana-v さんの提案により、潜在的な問題点が浮き彫りになり、その重要性を検討できました。ありがとうございました!
本 issue は NoGo につき close とします。

@tarepan tarepan closed this as not planned Won't fix, can't repro, duplicate, stale Mar 17, 2024
@tarepan tarepan removed the 要議論 実行する前に議論が必要そうなもの label Mar 17, 2024
@tarepan tarepan removed their assignment Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants