-
Notifications
You must be signed in to change notification settings - Fork 123
[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
[Rust]デプロイ処理を実装した #196
Conversation
onnxruntimeも一緒に出力するようにした
main元にcodesign処理追加してみましたが私の環境では確認はできなさそうです |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PRありがとうございます!!
マージされ次第一度releaseを作ってみようと思います。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
デプロイ処理実装ありがとうございます! 2つコメントしました。ご確認いただければ幸いです。
use_gpuについてもcuda制御のためのフラグだったのでuse_cudaに変更した
これなんですが、 |
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
情報ありがとうございます! |
- os: ubuntu-latest | ||
feature: default | ||
target: x86_64-unknown-linux-gnu | ||
use_gpu: true | ||
artifact_name: linux-x64-cuda |
There was a problem hiding this comment.
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
に変えた感じでしょうか?(個人的には確かに変更後の方が統一感があってスッキリしますが……)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あ、今のcoreのリリースはlinux-x64-gpuなんですね!!
There was a problem hiding this comment.
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にしたほうが良いと思いました
onnxruntimeのコピー処理macosかそうでないかで分けてましたが、実行を || trueにしておけばどのOSでも一まとまりにできそうなので短縮しました |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
なんとかしてrelease作ってみます。
* デプロイ処理を実装した 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>
内容
onnxruntimeも一緒に出力するようにした
関連 Issue
refs #128