Skip to content

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Apr 5, 2025

内容

一旦 #709 は抜きで、 コア0.16の導入を行う。

関連 Issue

Fixes: #1659
Resolves: #1571

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

その他

@qryxip qryxip marked this pull request as ready for review April 20, 2025 18:08
@qryxip qryxip requested a review from a team as a code owner April 20, 2025 18:08
@qryxip qryxip requested review from Hiroshiba and removed request for a team April 20, 2025 18:08
@qryxip
Copy link
Member Author

qryxip commented Apr 20, 2025

とりあえず後回しにしていたCUDA & cuDNN関係を片付けたのでdraft外しました。
あとはこのPRの状態のENGINEがちゃんと動くか確認するという流れになると思います。

@Hiroshiba
Copy link
Member

とりあえず後回しにしていたCUDA & cuDNN関係を片付けたのでdraft外しました。
あとはこのPRの状態のENGINEがちゃんと動くか確認するという流れになると思います。

ありがとうございます!!!!!!!

方針3つあるんですけど、どうしましょうか。

1つ目が @qryxip さんのところでビルドしていただく感じです。
普通にworkflow_dispatchになってるんで、どこでもビルドできるはずではあります。
どこかorg前提になってるところはなきにしもあらずかも・・・。

2つ目が一旦VOICEVOX側でproject-core-0.16みたいなのを作って、そちらにマージしてからビルドする流れです。
みんな試せるっちゃ試せるけどひと手間かかるので一長一短。

3つ目がえいやでmainブランチにマージする流れです。
できればこれは念のために避けたいかもです。
しばらくリリースの予定はありませんが、念のために1段階挟みたいかも。

まあでもVOICEVOX ENGINE側で課題を握れた方がいいので、2番目が一番まるそうな気がしてきました!
とりあえずレビューしてみます!!

@qryxip
Copy link
Member Author

qryxip commented Apr 20, 2025

1つ目が @qryxip さんのところでビルドしていただく感じです。

今開始してみました。
https://github.com/qryxip/voicevox_engine/actions/runs/14562288855

前は駄目だったのですが、$GITHUB_TOKENの権限がread onlyのままだったようで書き込み権限を与えたので大丈夫なはず。

@qryxip
Copy link
Member Author

qryxip commented Apr 25, 2025

d7ffcf4 (#1587)
#1659 の調査結果から、とりあえずWindowsをwindows-2022にしました。
test-engine-packageまで無事に通るようになっています。
https://github.com/qryxip/voicevox_engine/actions/runs/14669803025

@qryxip
Copy link
Member Author

qryxip commented Apr 26, 2025

とりあえず

方針3つあるんですけど、どうしましょうか。

1つ目が @qryxip さんのところでビルドしていただく感じです。

に話を戻すと、私のリポジトリで999.999.202504260207をビルドしてみました。次の5つを確認できれば、ビルドがちゃんとできているということになると思います。

[追記] 999.999.202504281454をビルドしました。

  • x86_64 Windows DirectML
  • x86_64 Windows NVIDIA GPU
  • x86_64 Linux NVIDIA GPU
  • Dockerfile (TARGETPLATFORM=linux/amd64, USE_GPU=false)
  • Dockerfile (TARGETPLATFORM=linux/amd64, USE_GPU=true)
  • Dockerfile (TARGETPLATFORM=linux/arm64/v8, USE_GPU=false)

Suggested-by: sabonerune <102559104+sabonerune@users.noreply.github.com>
Refs: VOICEVOX#1659 (comment)
@qryxip qryxip requested review from Hiroshiba and removed request for Hiroshiba August 14, 2025 14:45
@qryxip
Copy link
Member Author

qryxip commented Aug 14, 2025

COREのバージョンを0.16.1にしました。

ということで念のため、このPRブランチ自体でビルドしてみます。
(動作確認は今日やったのでやらなくてよさそう)

@Hiroshiba Hiroshiba requested a review from Copilot August 16, 2025 10:51
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades the core components to version 0.16, updating CUDA, ONNX Runtime, and related dependencies with improved download mechanisms and library organization.

  • Updates VOICEVOX core from version 0.15.9 to 0.16.1
  • Upgrades CUDA Toolkit from 11.8.0 to 12.8.1 and cuDNN from 8.9.2 to 8.9.7
  • Replaces batch downloads with individual component downloads using the new downloader capabilities

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
tools/licenses/cuda/EULA.txt Updates CUDA license text to version 12.8.1 with revised terms and new components
tools/generate_licenses.py Updates version numbers for ONNX Runtime, CUDA Toolkit, and cuDNN dependencies
.github/workflows/build-engine.yml Refactors build workflow to use individual component downloads and updates library paths

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

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!!

ちょっとこちらでsuggest適用して変更させていただきます!

(以下ただのコメントです)

今はmodelsやCOREやadditional_libraryというアセットに対して

  1. 全アセットのキャッシュを確認
  2. アセットAのダウンロード
  3. アセットAのキャッシュ保存
  4. アセットBのダウンロード
  5. アセットBのキャッシュ保存
  6. ...

ってなってるけど、たぶんキャッシュ確認→ダウンロード→キャッシュ保存をアセットごとにやったほうが綺麗そう。
でも差分が大きくなってしまうのでこのPRは今の形が良さそう!

@Hiroshiba Hiroshiba enabled auto-merge August 16, 2025 11:23
@Hiroshiba Hiroshiba added this pull request to the merge queue Aug 16, 2025
@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 16, 2025

📝 念の為にマージ後にpreview版をビルドしてエディタの方にも適用してビルドしたのを動かしてみる

ビルドはこれ
https://github.com/VOICEVOX/voicevox_engine/releases/tag/0.25.0-preview.2

Merged via the queue into VOICEVOX:master with commit 66818e8 Aug 16, 2025
5 checks passed
@qryxip qryxip deleted the pr/build-use-core-0-16 branch August 16, 2025 11:39
@Hiroshiba
Copy link
Member

こちらを組み込んだエディタを起動して、ちゃんと問題なく動くことを確認しました!!
modelディレクトリにもちゃんとvvmが入ってそう!! 🙌

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.

GHAでビルドしたWindows版エンジンがonnxruntime-builder製ORTを読み込めない コアを0.16にする
3 participants