-
Notifications
You must be signed in to change notification settings - Fork 2
chore: fill OrtTrainingApi
with v1.17's training API
#14
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
chore: fill OrtTrainingApi
with v1.17's training API
#14
Conversation
Co-authored-by: Carson M. <carson@pyke.io>
うーん。テスト用に外からダウンロードしてたっぽいmnist.onnxが404になったことで、一部は通っていたCIがめでたく全滅ですか。 まあ以前も言った気がしますが放置でよさそう。CIをどうにかしようとするより、v2.0.0-rc.9 (or v2.0.0-rc.10?)の追従を入れる方が真っ当だし多分早い。 |
よくわかってないのですが、既存のワーニングを直したところ、それとは関係なくエラーが見つかったって感じでしょうか? であれば問題なさそう! (以下ついでに) |
このリポジトリ自体(ort)の方このリポジトリ自体(ort)については、CORE v1.16を早くリリースしましょうという話の最中だったので単に後回しにされていた、という認識です。ONNX Runtime自体はv1.17のまま最新のpykeio/ortに追従するといったことも普通にできるはずで、この #14 はその予行演習でもありました。 (VOICEVOX) ONNX Runtimeの方VOICEVOX/voicevox_core#888 とか VOICEVOX/voicevox_core#968 みたいな話も解決されるかもなので、(VOICEVOX) ONNX Runtimeの方についても最新版の追従はやっていきたくはありますね。
1.18.0用リリースブランチには入ってますが、まだリリースされていないという状態ですね…。そろそろリリースがされそうな雰囲気は漂っているのですが、ONNX Runtime側に導入されてそっちのリリースも行われるまで時間はかかりそう。
それは朗報…だと思うのですが、他の選択肢もあるので悩むところではありますね。
|
なーるほどです! vocevox_onnxruntimeのバージョンに関してはvoicevox_onnxruntime側に議論切った方がわかりやすいかも? 後で聞くことメモ: |
リリースされそうな雰囲気がすると言っていたらonnx v1.18.0がつい今日出ましたね。あとどうもONNX Runtime側でもmainブランチでは実は先んじて microsoft/onnxruntime#24449 ということは行われていていたらしく、Yosshiさんのやつも入っているっぽい。なのでONNX Runtimeのmainブランチであれば [追記] ONNX Runtimeの今のmainブランチをv1.23.0-alpha.202505122057みたいに勝手に名付けてしまって使うという手もありそう (ちなみにmicrosoft/onnxruntime#24449は結構ややこしいことをしてそうだったので、これをONNX Runtime v1.22.0とかにバックポートするというのは得策ではなさそう。それをやるくらいなら名無し。さんがやってたようにYosshiさんのパッチをピンポイントで当てた方がよさそう) ともあれ、issueは作ろうと思います。ort側で対処すべき問題はそんなに無いはずなので、onnxruntime-builderですかね? |
onnxruntimeのリリーススケジュールがそんなに長期スパンじゃなさそうで、かつすぐにリリースしたいorアプデする必要がなければ、3が良さそうなのかなと思いました! |
内容
To make
cargo clippy --workspace --all-targets
pass.(though we don't use training API)
Copied
enum OrtPropertyType
from pykeio#202.関連 Issue
スクリーンショット・動画など
その他