Skip to content

LinuxビルドをPyInstallerへ移行し、他OSとビルドジョブを統合する #450

Merged
y-chan merged 18 commits intoVOICEVOX:pyinstallerfrom
y-chan:feature/pyinstaller-linux
Aug 15, 2022
Merged

LinuxビルドをPyInstallerへ移行し、他OSとビルドジョブを統合する #450
y-chan merged 18 commits intoVOICEVOX:pyinstallerfrom
y-chan:feature/pyinstaller-linux

Conversation

@y-chan
Copy link
Copy Markdown
Member

@y-chan y-chan commented Aug 14, 2022

内容

題の通り
一応CPU版の動作確認はしましたが、持っている機器の関係上GPUが正常に使えるか確認できていません。
以下にGitHub Actions上でビルドしたものをGoogle Driveに上げておいた(GitHub ActionsのArtifactsは他人は確認できないため)ので、どなたか確認していただけると助かります。
https://drive.google.com/file/d/1d6JXgowXtv-5RUbmtuWt6rhdDr8INTcc/view?usp=sharing

関連 Issue

ref #439

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

Artifacts自体は300MBぐらい軽量化できました

PyInstaller Nuitka
image スクリーンショット 2022-08-14 11 42 12

ビルド時間は全体で10分程度になっています
image

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 14, 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 199 159 coverage-20%
voicevox_engine/synthesis_engine/make_synthesis_engines.py 48 39 coverage-19%
voicevox_engine/synthesis_engine/synthesis_engine.py 133 12 coverage-91%
voicevox_engine/synthesis_engine/synthesis_engine_base.py 66 9 coverage-86%
voicevox_engine/user_dict.py 131 10 coverage-92%
voicevox_engine/utility/init.py 3 0 coverage-100%
voicevox_engine/utility/connect_base64_waves.py 35 3 coverage-91%
voicevox_engine/utility/engine_root.py 9 2 coverage-78%
TOTAL 1203 248 coverage-79%

@qryxip
Copy link
Copy Markdown
Member

qryxip commented Aug 14, 2022

nvidia機でLinuxを使っています。

確認すべき箇所がよくわかりませんが、単純に短いテキストをsynthesisする分には--use_gpuで動作しました。

@qryxip
Copy link
Copy Markdown
Member

qryxip commented Aug 14, 2022

長文を入れても大丈夫でした。

@y-chan
Copy link
Copy Markdown
Member Author

y-chan commented Aug 14, 2022

use_gpuでsynthesisできていれば大丈夫です!確認ありがとうございます!

Copy link
Copy Markdown
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.

とても見通しが良くなったと思います、素晴らしいです!!!!

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
@y-chan
Copy link
Copy Markdown
Member Author

y-chan commented Aug 14, 2022

コメント頂いた件適用してみました...!
多分大丈夫だと思います...!

@Hiroshiba
Copy link
Copy Markdown
Member

LGTM!! 更に見通しが上がりましたね・・・!
マージしていただいても大丈夫です!

dockerに関してはいかがでしょう?👀
(linuxビルドしてできたrun.distディレクトリをdockerに全部addする形式だとかなり綺麗に書けそう?)

@y-chan
Copy link
Copy Markdown
Member Author

y-chan commented Aug 15, 2022

Dockerの方も取り掛かりたいですね...!
あと、MacのUniversal2対応も手をつけたいかも...👀

とりあえずこちらのPRはマージしてしまいますね!

@y-chan y-chan merged commit 54357d4 into VOICEVOX:pyinstaller Aug 15, 2022
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