コアのctypes実装の修正など

[takana-v]

内容

現在のctypes実装ではファイル名に基づいたファイル探索を行っていますが、0.10の製品版に対応していません。
これを修正します。

• dllの場所がフォルダ直下からcore/lib以下に移動
  • --voicevox_dirの修正
  • binファイルとdllの場所が違う可能性を考慮する必要が生じた
• dllの名前にアーキテクチャ名が含まれずcore.dllのまま
  • dllの名前でのコアの種類の判別が不可能になった

実現方法

voicebin_dir（仮称）を作り、binファイル群の場所を指定するようにする
voicebin_dirからmetas.jsonを読み込み、バージョンを確認する

できれば同時にこのリファクタリングも行いたいです。
#254 (comment)

その他

他の人と被らないようにするためのIssueです。

[Hiroshiba]

issue化ありがとうございます！

voicebin_dir

こちら、役割的にはvoicelib_dirsと同じなのかなと思ったのですがどうでしょう👀

voicevox_engine/voicevox_engine/synthesis_engine/make_synthesis_engines.py

Lines 26 to 27 in c7cda72

	voicelib_dirs: List[Path], optional, defauld=None
	音声ライブラリ自体があるディレクトリのリスト

[takana-v]

現状ではvoicelib_dirsに「core.dllの場所」と「*.binの場所」の2つの意味があります。
（initializeの時に後者の意味で使っています）

これまではそれでも問題ありませんでしたが、0.10でdllとbinの場所が分かれたので、「*.binの場所」を示す、voicebin_dirを作りたいと思っています。
（もう少しいい名前があればそれにしたいです）

（追記）

voicevox_engine/voicevox_engine/synthesis_engine/make_synthesis_engines.py

Line 68 in c7cda72

voicelib_dirs = [p.expanduser() for p in voicelib_dirs]

ここの部分でのvoicelib_dirsです。

[Hiroshiba]

なるほどです！！！
バージョン0.10のエンジンパッケージは理由があってcore/lib/core.dllにあるんですね！！

https://github.com/VOICEVOX/voicevox_engine/blob/release-0.10/.github/workflows/build.yml#L759-L761

今のmasterだと、バージョン0.10をvoicevox_dirに与えるとうまく動かないから、voicebin_dirを追加して修正する、という認識であっていますか👀

であれば、今のmasterはバージョン0.10のエンジンをvoicevox_dirで指定できない、という仕様にしてしまう手もあるかなと思いました。
というのも、現状のディレクトリ構成は結構複雑で、これ以上増やすとメンテナンスコストが上がってしまう一方、0.11に上がればおそらく不要になってしまうためです。

今のmasterで0.11プレビュー版をビルドし、それをvoicevox_dirに指定してもらうことで問題を迂回できるのであればそうしたいのですが、どうでしょうか👀

[takana-v]

それで問題ないかと思います。
タスクはこのような感じでしょうか。
とりあえずこのIssueはcloseしておきます。

• 0.11プレビュー版をビルド
• 0.10では動かない旨をIssueでピン止め

[Hiroshiba]

0.10で動かない件はREADMEに書く手も良いかもです。

0.11のプレビュー版をビルドし始めてみました！
https://github.com/VOICEVOX/voicevox_engine/releases/tag/0.11.preview-0