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

C API用のzipファイルからmodelディレクトリをなくし、別でmodelダウンロード可能にする #596

Closed
Hiroshiba opened this issue Sep 1, 2023 · 3 comments · Fixed by #603

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Sep 1, 2023

内容

C API用のdllが入ったZIPファイルの中には、音声合成するためのモデルが大量に含まれている700MB程度のmodelディレクトリが同梱されています。
dllが含まれているZIPファイルからこのmodelディレクトリをなくし、別でダウンロード可能にしておくと色々都合が良さそうです。

例えばこちらの課題ある通り、全部のモデルをダウンロードするのではなく、必要なモデルだけをダウンロードする需要があったりします。
これを実現するためには、とりあえずまずはZIPファイルの中からモデルディレクトリを無くす必要がありそうです。

Pros 良くなる点

モデルファイルを別途ダウンロード可能になる。
OSや環境ごとにモデルの同梱が不要になるのでビルドが早くなる。

Cons 悪くなる点

設計が自明ではない。

実現方法

色々やり方がありそうですが、変更量が小さくなるようにまずは以下の流れを考えてます。

  1. dll用のZIPにモデルディレクトリを同梱しないようにする
  2. 別でモデルディレクトリをZIPファイルとしてリリースする
  3. ダウンローダー内でモデルディレクトリZIPファイルをダウンロード&展開可能にする

この形であれば、ZIPファイルダウンロード&ダウンローダーを実行した結果が、変更前と変更後で同じになると思ってます。

その他

0.15リリースに間に合わせられると良さそうかも。

ちなみにPython wheelの中からモデルディレクトリがすでに消えているっぽい・・・?

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Sep 1, 2023

@qryxip みたいなこと考えているんですがどう思われますか・・・? 👀
多分発展系として色々できそうではあるのですが、とりあえず第1弾として分離だけやっちゃおうかなと。
問題なさそうだったら取り組んでみようと思ってます!

@qryxip
Copy link
Member

qryxip commented Sep 2, 2023

ひとまず分離するというのはよいと思います。
(SHAREVOXも分離された状態でReleaseに上がってますね)

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Sep 2, 2023

メモです

  • READMEの分離どうするか
  • 用語
    • コア内ではVVMを音声モデルと呼んで案内する
      • 適宜「音声モデル(音声ライブラリ)」と記載して語弊をなくす
    • この用語集に従う
    • コアの中で音声モデルと呼んでいることと、他の場所では音声ライブラリと呼んでいることがややこしくないか・異常が起きていないかを確認する必要がある
  • 案内
    • releases内の案内調整
      • 「example見てね」が安泰かも
    • 各exampleの案内調整

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants