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

複数エンジン対応:vvppファイルのビルドを追加 #524

Merged
merged 9 commits into from
Dec 10, 2022

Conversation

sevenc-nanashi
Copy link
Member

内容

vvppファイルのビルドを追加します。
サンプルビルド:sevenc-nanashi:0.14.0-preview-vvpp.3

関連 Issue

スクリーンショット・動画など

(なし)

その他

(なし)

@github-actions
Copy link

github-actions bot commented Dec 7, 2022

Coverage Result

Resultを開く
Name Stmts Miss Cover
voicevox_engine/init.py 1 0 coverage-100%
voicevox_engine/acoustic_feature_extractor.py 75 0 coverage-100%
voicevox_engine/dev/synthesis_engine/init.py 2 0 coverage-100%
voicevox_engine/dev/synthesis_engine/mock.py 36 2 coverage-94%
voicevox_engine/full_context_label.py 162 3 coverage-98%
voicevox_engine/kana_parser.py 86 1 coverage-99%
voicevox_engine/model.py 154 7 coverage-95%
voicevox_engine/mora_list.py 4 0 coverage-100%
voicevox_engine/part_of_speech_data.py 5 0 coverage-100%
voicevox_engine/preset/Preset.py 12 0 coverage-100%
voicevox_engine/preset/PresetLoader.py 34 1 coverage-97%
voicevox_engine/preset/init.py 3 0 coverage-100%
voicevox_engine/synthesis_engine/init.py 5 0 coverage-100%
voicevox_engine/synthesis_engine/core_wrapper.py 206 166 coverage-19%
voicevox_engine/synthesis_engine/make_synthesis_engines.py 57 49 coverage-14%
voicevox_engine/synthesis_engine/synthesis_engine.py 133 12 coverage-91%
voicevox_engine/synthesis_engine/synthesis_engine_base.py 67 9 coverage-87%
voicevox_engine/user_dict.py 129 6 coverage-95%
voicevox_engine/utility/init.py 3 0 coverage-100%
voicevox_engine/utility/connect_base64_waves.py 37 0 coverage-100%
voicevox_engine/utility/path_utility.py 26 6 coverage-77%
TOTAL 1237 262 coverage-79%

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@Hiroshiba
Copy link
Member

vvppを分割にしておく理由をgithub上にメモされてなさそうでした。

  1. github releaseasでは2GBを超えるファイルをアップロードできない
  2. エディタのvvpp読み込みは分割vvppに対応していないけど、将来的に対応したいかも

みたいな感じでしたっけ。

@Hiroshiba
Copy link
Member

@aoirint さん、vvppは複数エンジン対応用途なのですが、レビューお願いしても大丈夫でしょうか 👀

vvppはVOICEVOX ENGINE Plugin Packageの略で、エンジンディレクトリをzipファイルに固めて、.zipを.vvppにリネームしたものです。
ファイル分割の意図は↑のコメントのとおりです。

まだvvppロード側のPRを導入したエディタのリリースを作れてないのでテスト読み込みはできないのですが、レビューお願いできると心強いです!!

@Hiroshiba Hiroshiba requested a review from aoirint December 7, 2022 17:16
@sevenc-nanashi
Copy link
Member Author

sevenc-nanashi commented Dec 7, 2022

  1. github releaseasでは2GBを超えるファイルをアップロードできない

  2. エディタのvvpp読み込みは分割vvppに対応していないけど、将来的に対応したいかも

そんな感じですー。2の分割読み込みは VOICEVOX/voicevox#991 がマージされたらやろうと思います。(PRが巨大化しているので)

@sevenc-nanashi
Copy link
Member Author

レビューの部分を修正しましたー。
この機会に7zも001だけなら数字部分を消してもいいかもしれない?

@Hiroshiba
Copy link
Member

7zの方は今の形に依存してるサードパーティアプリあるかもなので、とりあえずそのままで…!

Copy link
Member

@aoirint aoirint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@aoirint aoirint merged commit b28601b into VOICEVOX:master Dec 10, 2022
@aoirint
Copy link
Member

aoirint commented Dec 10, 2022

メモです。

zipファイルの分割方法ですが、splitコマンドを使ってファイルを分割した場合、cat archive.001.zip archive.002.zip > archive.zipのように、単純にバイナリを連結すると展開することができる方式になっています(分割ファイルごとにヘッダなどは設けず、バイナリを切り取っただけ)。

ほかに、zipsplitというコマンドがあって、こちらは分割したファイル1つ1つが単体でzipとして機能するようです。
しかし深層学習モデルやCUDAライブラリなど容量の大きいファイルの同梱が想定され、1ファイルで分割単位を超える可能性があるため、splitコマンドの方がいいなと思いました。

7zの分割については、VOICEVOX ENGINEではp7zipの機能を使って分割していますが、ファイルサイズが分割単位と一致するので、splitと同じことをしていそうだなと思いました(特殊なヘッダがあるようなものではなさそう)。実際、catで連結したものをWindowsの7-zip GUIで開くことができそうです。

ちなみに複数エンジンとは関係ないですが、VOICEVOXエディタのWindows nsis用7zファイルはsplitを使って分割していて、p7zipによる分割と命名規則がずれてしまっているので、可能ならちょっと揃えたい気もします...(拡張子を.nsis.7z.0 -> .nsis.7z.001.nsis.7z.1 -> .nsis.7z.002に変えることができると、Windowsの7-zip GUIで開くことができそう)

@Hiroshiba
Copy link
Member

すみません、プレビュー版の作成を忘れていました!
こちらで作成中です。
https://github.com/VOICEVOX/voicevox_engine/releases/tag/0.14.0-preview.5

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

Successfully merging this pull request may close these issues.

3 participants