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.

Sign up for GitHub

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

Jump to bottom

ビルド時のコアのバージョンを 0.12.0-preview.3 に上げる #401

Merged
merged 19 commits into from
May 14, 2022
Merged

ビルド時のコアのバージョンを 0.12.0-preview.3 に上げる #401

merged 19 commits into from
May 14, 2022

Conversation

PickledChair
Copy link
Member

@PickledChair PickledChair commented May 12, 2022 •
edited
Loading

内容

タイトルの通りです。

コアのバージョン 0.12.0-preview.2, 0.12.0-preview.3 で以下の変更が加えられました：

  • voicevox_core リポジトリで提供されている、バイナリを含む zip ファイルがアーキテクチャごとに分かれるようになった
  • 共有ライブラリの名称が全てのアーキテクチャで統一 (Windows: core.dll, Linux: libcore.so, macOS: libcore.dylib) されるようになった
  • metas.json やモデルファイルが共有ライブラリに埋め込まれるようになった

これらの変更に対応します。

テストリリースはこちらです: https://github.com/PickledChair/voicevox_engine/releases/tag/0.12.3-pickledchair-preview.6

関連 Issue

close #396

@github-actions
Copy link

github-actions bot commented May 12, 2022 •
edited
Loading

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 40 4 coverage-90%
voicevox_engine/experimental/init.py 0 0 coverage-100%
voicevox_engine/experimental/guided_extractor.py 128 94 coverage-27%
voicevox_engine/experimental/julius4seg/init.py 0 0 coverage-100%
voicevox_engine/experimental/julius4seg/converter.py 298 295 coverage-1%
voicevox_engine/experimental/julius4seg/sp_inserter.py 116 89 coverage-23%
voicevox_engine/full_context_label.py 162 3 coverage-98%
voicevox_engine/kana_parser.py 86 1 coverage-99%
voicevox_engine/model.py 147 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 182 145 coverage-20%
voicevox_engine/synthesis_engine/make_synthesis_engines.py 52 43 coverage-17%
voicevox_engine/synthesis_engine/synthesis_engine.py 181 69 coverage-62%
voicevox_engine/synthesis_engine/synthesis_engine_base.py 68 9 coverage-87%
voicevox_engine/user_dict.py 107 10 coverage-91%
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 1755 775 coverage-56%

PickledChair added 11 commits May 13, 2022 00:15
@PickledChair
共有ライブラリの名称変更やmetas.json・モデルファイルがなくなったことを反映
08ec35b
@PickledChair
VOICEVOX_CORE_ASSET_NAMEの定義場所を変更
8a12fd4 
env.VOICEVOX_CORE_VERSIONを認識してくれなかったため
@PickledChair
trivial change
2a6e7dc
@PickledChair
Merge remote-tracking branch 'VOICEVOX/voicevox_engine/master' into b…
530c13f 
…uild-with-core-version-0-12-0-preview-3
@PickledChair
zip解凍後のフォルダ名を正しく指定する
77cb4d4
@PickledChair
glob パターンはダブルクォートの外に出さないと機能しなかったので修正
5f5e426
@PickledChair
coreのアセットのzip解凍をunzipではなく7zで行う
328fa1d
@PickledChair
VERSIONファイルの移動を復活
b354fcf
@PickledChair
現在のアセットにはREADMEが含まれていないのでREADMEのcatを削除
ed0b12e
@PickledChair
Windows版におけるコアのzip解凍方法に関するコメントを追加
4b65a33
@PickledChair
build-docker.ymlをコア0.12.0-preview.3に対応させる
ed2549a
@PickledChair PickledChair marked this pull request as ready for review May 12, 2022 21:02
PickledChair
PickledChair commented May 13, 2022
View reviewed changes
voicevox_engine/synthesis_engine/core_wrapper.py
Comment on lines +295 to +299
# NOTE: CDLL クラスのコンストラクタの引数 name には文字列を渡す必要がある。
# Windows 環境では PathLike オブジェクトを引数として渡すと初期化に失敗する。
return CDLL(
str((core_dir / CORENAME_DICT[platform.system()]).resolve(strict=True))
)
Copy link
Member Author

Choose a reason for hiding this comment

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

前に #388 を実装したときに、Windows 版の新しいコアが読み込めないバグを埋め込んでいたことに気づいたので、合わせて修正しました（最後のエンジンの実行テストに Windows 版だけ失敗したので気がつきました……）。

Hiroshiba
Hiroshiba approved these changes May 13, 2022
View reviewed changes
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！！

Dockerfile Show resolved Hide resolved
@Hiroshiba Hiroshiba mentioned this pull request May 13, 2022
Closed
@takana-v takana-v mentioned this pull request May 14, 2022
Merged
takana-v
takana-v reviewed May 14, 2022
View reviewed changes
Copy link
Member

@takana-v takana-v left a comment

Choose a reason for hiding this comment

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

READMEのビルドの項目も修正をお願いします。
それ以外は問題ないと思います。

@PickledChair
Copy link
Member Author

READMEのビルドの項目も修正をお願いします。

ご指摘ありがとうございます。修正しました！

takana-v
takana-v approved these changes May 14, 2022
View reviewed changes
Copy link
Member

@takana-v takana-v left a comment

Choose a reason for hiding this comment

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

LGTM！

@takana-v takana-v merged commit 01f1fd2 into VOICEVOX:master May 14, 2022
@PickledChair PickledChair deleted the build-with-core-version-0-12-0-preview-3 branch September 3, 2022 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@takana-v takana-v takana-v approved these changes

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ビルド周りをコア 0.12.0-preview.3に対応する
3 participants
@PickledChair @Hiroshiba @takana-v