VOICEVOX / onnxruntime-rs

Rust wrapper for Microsoft's ONNX Runtime
Apache License 2.0
6 stars 13 forks source link

tracing-subscriberをv0.3に上げる #22

Closed qryxip closed 1 year ago

qryxip commented 1 year ago

内容

tracing-subscriberをv0.2 → v0.3に上げることで、CIが謎に落ちている状態を解消します。 https://github.com/VOICEVOX/onnxruntime-rs/pull/21#issuecomment-1518912412

よくわかりませんがこれで直るようです。 https://github.com/qryxip/onnxruntime-rs/actions/runs/4775863128/jobs/8490490491

関連 Issue

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

その他

このリポジトリで使われている、tracing-subscriberを内部で使っているtest-env-logですが、"test-log"に改名したということでdeprecatedになっており、microsoft/onnxruntimeの方では既にそっちに切り替わってます。このライブラリがもしかしたら今回の問題の原因の一端なんじゃないかと疑っています。 ただ移行を考えるとこのリポジトリのは放置でいいかも。 いや#[deprecated]によるwarningも出てますし、Cargo.tomlを追加で一行変更するだけでいいので今やった方がいいですね。

Hiroshiba commented 1 year ago

PRありです!! マージします!!

qryxip commented 1 year ago

Rustのバージョンアップでどこか変わって、それにtracing-なんとかライブラリが追従したとかですかねぇ。

あーRustが1.69になってましたね。忘れてました。Rust本体の何をどう踏んづけたらあんなエラーになったのかは個人的に気になるところではありますが、まあ直ったし…