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

ビルド周りをコア 0.12.0-preview.3に対応する #396

Closed
Hiroshiba opened this issue May 10, 2022 · 5 comments · Fixed by #401
Closed

ビルド周りをコア 0.12.0-preview.3に対応する #396

Hiroshiba opened this issue May 10, 2022 · 5 comments · Fixed by #401
Assignees
Labels

Comments

@Hiroshiba
Copy link
Member

内容

コアが0.12.0-preview.2に上がったタイミングで、アーキテクチャごとにzipファイルが分かれるようになりました。

この分かれたzipにエンジンのビルドを追いつかせるのがこのissueの目的です。

Pros 良くなる点

アプデ出来るようになる

Cons 悪くなる点

実現方法

大きな差分は以下の3つになりそうです

  • ライブラリにバイナリが同梱されている
  • zipファイルが分かれている
  • ライブラリにアーキテクチャ名が含まれていない

build.ymlやbuild-docker.ymlの変更が必要だと思われます。

VOICEVOXのバージョン

0.?.0

その他

@Hiroshiba Hiroshiba added the 優先度:高 最優先 label May 10, 2022
@Hiroshiba
Copy link
Member Author

@PickledChair さんにお願いさせていただきました!!よろしくお願いします・・・!

@PickledChair
Copy link
Member

今気づいたのですが、0.12.0-preview.20.12.0-preview.3 で zip ファイルの構造が変わっている (VOICEVOX/voicevox_core#117) ので、0.12.0-preview.2 に対応させたあと、0.12.0-preview.3 以降に対応させるために再びビルド手順を微修正することになりそうです。

@Hiroshiba
Copy link
Member Author

おっと、なるほどです!
issueタイトルは.2になっていますが、いきなり.3対応でも全く問題ないです…!

@PickledChair
Copy link
Member

いきなり.3対応でも全く問題ないです…!

わかりました! ちなみに、0.12.0-preview.3 のコアを今のエンジンで読み込むと、遅延ロードの仕組みが加わった関係でモデルのロードがされず、音声の生成ができないことを確認しています。とりあえず initialize 関数の第3引数 load_all_modelsTrue を渡すだけで解決するのですが、一応 PR を分けて、先にこの引数変更の対応をしてからビルドの方に取り組みたいと思います。

@Hiroshiba
Copy link
Member Author

ありがとうございます!
coreに関しても助かります。

@PickledChair PickledChair changed the title ビルド周りをコア 0.12.0-preview.2に対応する ビルド周りをコア 0.12.0-preview.3に対応する May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants