8000 [Rust]デプロイ処理を実装した by qwerty2501 · Pull Request #196 · VOICEVOX/voicevox_core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Rust]デプロイ処理を実装した #196

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

Merged
merged 10 commits into from
Jul 23, 2022

Conversation

qwerty2501
Copy link
Contributor

内容

onnxruntimeも一緒に出力するようにした

関連 Issue

refs #128

onnxruntimeも一緒に出力するようにした
@qwerty2501 qwerty2501 changed the title デプロイ処理を実装した [Rust]デプロイ処理を実装した Jul 21, 2022
@qwerty2501
Copy link
Contributor Author

main元にcodesign処理追加してみましたが私の環境では確認はできなさそうです

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.

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

マージされ次第一度releaseを作ってみようと思います。

Copy link
Member
@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

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

デプロイ処理実装ありがとうございます! 2つコメントしました。ご確認いただければ幸いです。

@qwerty2501
Copy link
Contributor Author

@Hiroshiba

マージされ次第一度releaseを作ってみようと思います。

これなんですが、 workflow_run ( workflow_dispatch も?) は最低でも一度default branchにmergeされないと動作しないようです。(ネットでそんなこと書いてました)
そのためとりあえず動作したい確認したい場合は動作確認用のbranchを切って pushなりreleaseを設定する必要がありそうです
こんな感じに

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
@Hiroshiba
Copy link
Member

情報ありがとうございます!
たしかに・・・。一瞬rustをdefault branchにしちゃってからactionsを流してすぐ戻そうかな・・・。

- os: ubuntu-latest
feature: default
target: x86_64-unknown-linux-gnu
use_gpu: true
artifact_name: linux-x64-cuda
Copy link
Member

Choose a reason for hiding this comment

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

すみません、ちょっと細かい確認なのですが、こちらは linux-x64-gpu から意図的に linux-x64-cuda に変えた感じでしょうか?(個人的には確かに変更後の方が統一感があってスッキリしますが……)

Copy link
Member

Choose a reason for hiding this comment

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

あ、今のcoreのリリースはlinux-x64-gpuなんですね!!

Copy link
Contributor Author
@qwerty2501 qwerty2501 Jul 23, 2022

Choose a reason for hiding this comment

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

あー、linuxとWindowsでgpuかcudaで別れてるってことだったんですね
engineやeditorの変更にも影響与えそうなのでとりあえず元のものと合わせましたが、今後linuxでもcuda以外のgpu実装が出る可能性があるので然るべきタイミングでcudaにしたほうが良いと思いました

@qwerty2501
Copy link
Contributor Author

onnxruntimeのコピー処理macosかそうでないかで分けてましたが、実行を || trueにしておけばどのOSでも一まとまりにできそうなので短縮しました

Copy link
Member
@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

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

LGTM!

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

なんとかしてrelease作ってみます。

@Hiroshiba Hiroshiba merged commit bdb1894 into VOICEVOX:rust Jul 23, 2022
qwerty2501 added a commit to qwerty2501/voicevox_core that referenced this pull request Jul 23, 2022
* デプロイ処理を実装した

onnxruntimeも一緒に出力するようにした

* codesign処理追加

* /usr/bin/env に変更

* 不足していた変数定義を行った

* gpu->cudaに変更した

use_gpuについてもcuda制御のためのフラグだったのでuse_cudaに変更した

* core.libをコピーする処理を追加

refs VOICEVOX#196 (comment)

* codesign.bashに名称変更して元のscript内容と同じにした

* Update .github/workflows/build_and_deploy.yml

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>

* linuxのcuda番のartifact_nameをgpuに変更した

* onnxruntimeのコピー処理を短縮化

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
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
0