-
Notifications
You must be signed in to change notification settings - Fork 309
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
Migrate: vue-cliからviteに移行 #1207
Conversation
良いですね!!! |
そういえば: |
テスト・Lintまで動いたのでDraft外そうと思います。 テストビルド: https://github.com/sevenc-nanashi/voicevox/actions/runs/4202793119 |
デフォルトで存在していたものなのでわからないです 😇 |
このプロジェクトは |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ですね。 |
speakerIdの失敗ぽいですね、エンジンの話者読み込みで死んでる感…? |
もしかしたら製品版のconfig.jsonがなんらかの原因でマイグレーションできてないのかなと思いました。 エラーになるconfig.jsonはぜひ精査したいので、なんか適当にリネームして実行してみると良いかも・・・? |
確認したところ、確かにある意味当たっていました。 ただし問題の見つかった config.json は vite 版 VOICEVOX が新しく生成した辞書の方でした。 "defaultStyleIds": [
{
"speakerUuid": "7ffcb7ce-00ec-4bdc-82cd-45a8889e43ff",
"defaultStyleId": 2,
"engineId": "00000000-0000-0000-0000-000000000000"
},
{
"speakerUuid": "388f246b-8c41-4ac1-8e2d-5d79f3ff56d9",
"defaultStyleId": 3,
"engineId": "00000000-0000-0000-0000-000000000000"
},
{
"speakerUuid": "35b2c544-660e-401e-b503-0e14c635303a",
"defaultStyleId": 8,
"engineId": "00000000-0000-0000-0000-000000000000"
},
...
] |
もう少し調べたのですが、デフォルトの 一方、「データ準備中」のままになる他の要因が分かりました。複数エンジン機能をオンにして使用しているのですが、vvpp ではなく「既存エンジン」を追加している場合( |
おおおおお・・・ なんか謎の挙動がありそうですね・・・。 |
とりあえずzodエラーの方はマイグレーション周りだろうなと思いました。 Lines 504 to 512 in e17daec
↓ここでDEFAULTのが入る・・・ Lines 145 to 152 in a9b65c6
なぜDEFAULTのエンジンIDが入ってないのかは不明ですが、こちらは開発版で起動したことがあるからなエラーな気がしました。 |
なるほどです。ちなみに、バージョン 0.14.3 の VOICEVOX でその |
0.14.3の場合、起動後にデフォルトスタイルIDが上書きされてるのかなと思います。 そもそも |
とりあえず時系列順に config.json を並べてみました。 その過程で気づいたのですが、2番めの vite 版 VOICEVOX が作成した config.json は (ただ、1番めの config.json で起動しないという問題は相変わらずあります。こちらは 追加の条件としては、最初は vvpp 指定の SHAREVOX と既存エンジン指定の COEIROINK がマルチエンジンとしてありました。2番目以降の config.json ではデフォルト設定の自動生成の際に vvpp の方だけが検出されて SHAREVOX エンジンだけが追加されている状態になっています。 1. 最初に vite 版 VOICEVOX で開いた直後の状態の config.json(ずっとデータ準備中になる)
2. 一度 config.json -> config-backup.json にリネームした時に vite 版 VOICEVOX が生成した config.json(`defaultStyleIds` は空になる。音声合成等ができるようになった)
この後、vite 版 VOICEVOX において再び COEIROINK を既存エンジンとして追加して再起動しましたが、再び「データ準備中」から動かなくなりました。COEIROINK を複数エンジンの設定から削除するとまた使えるようになりました。 |
0.14.4 で新しく生成した config.json の状態で COEIROINK を既存エンジンとして追加すると、0.14.4 では使えるが vite 版 VOICEVOX では「データ準備中」のままになる、ということも確認しました。こちらも
という指摘の通り、config.json に書いてある また、このことは開発版の config.json の内容を引き継いでいなくてもエラーになることを示しています。 |
開発ビルドでSharevoxをロードしてみました、普通に動きました。 |
確認してみましたが、マルチエンジンとして読み込んでいる COEIROINK(1.7.0 開発版)は MYCOE を導入していないですね……強いて言えば公式キャラの MANA さんの追加ダウンロード(MYCOE ではない)はしていると思います。 つまりつくよみちゃんと MANA さんだけがいます。#1207 (comment) の 1. の config.json の SHAREVOX について、vvpp だけでなく既存エンジン読み込みの方法も試してみましたが、こちらはおっしゃる通りどちらもうまく行きました。なので、確かに COEIROINK 側の問題かもしれませんね。ただし 0.14.4 では COEIROINK も既存エンジンとして追加して使えているので、VOICEVOX 側の仕様もどこかがある段階で変わり、それとの組み合わせで起動しなくなったように見えます。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ほぼLGTMです!
1点だけコメントつけましたが、正直どっちでもいいかも...?
ZodError
については、Vite化の影響ではないことがわかったので、 #1218 にて引き続き議論しましょう...!
僕としては @y-chan さんのコメントのとこと、あと↓が気になっているくらいです! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!
たび重なる調整、お疲れさまでした!!!
マージ後にpreviewビルドを作ってみます!
内容
vue-cliからviteに移行します。
関連 Issue
close: #1000
スクリーンショット・動画など
(なし)
その他
結構大きな変更になると思います。