Skip to content

Conversation

Hiroshiba
Copy link
Member

内容

サンプルのonnxモデルをログが大量に出ないものに置き換えます。

の副産物です。

その他

Copy link
Member

@qryxip qryxip left a comment

Choose a reason for hiding this comment

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

私の手元ではPython APIが動いています。何もわかりませんがLGTM

@Hiroshiba
Copy link
Member Author

確認ありがとうございます!!
大丈夫だと思うのでマージします…!

@Hiroshiba Hiroshiba merged commit eabad00 into main Feb 4, 2023
@Hiroshiba Hiroshiba deleted the ログが大量に出ないonnxモデルに変更 branch February 4, 2023 19:24
@qryxip qryxip mentioned this pull request Feb 10, 2023
3 tasks
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 10, 2023
Hiroshiba added a commit that referenced this pull request Apr 8, 2023
* `test_util`をクレートに分離する

* テスト機構を作る

* Actions上で動かす

* chore: `assert` → `assert_output`

* assert_cmdのエラーは`libtest_mimic::Failed`経由で表示する

* 得られた値は`assert_eq!`でチェックする

* `metas()`をテスト

* `yukarin_s_forward`まで

* 相対パスで十分

* `cargo build -p voicevox_core_c_api --lib`を単体で実行

* Revert "`cargo build -p voicevox_core_c_api --lib`を単体で実行"

This reverts commit 6473c8d.

* `onnxruntime/disable-sys-build-script`をオンにする

* Revert "`onnxruntime/disable-sys-build-script`をオンにする"

This reverts commit dbc99c9.

* `ORT_OUT_DIR=`にする

* 不要な`-p`を削除

* #423 に対応

* `list_windows_video_cards`の結果をマスク

* Minor refactor

* `cargo build`のスキップについてコメント

* `voicevox_get_version`を削除

* Minor refactor

* JSONであることはチェック

* 可能な限り"voicevox"の名前を排除

* `ORT_OUT_DIR=`についてコメント

* `$ORT_USE_CUDA`も無効化

* `cargo build`終了時点でDLLがあるかチェック

* Rust APIのテストを持って来る

* 正規表現を修正

* ActionsのWindows上でのみSHA-256はチェックする

* `decode_forward`のSHA-256を更新

* コメント

* モジュールを整理

* `Sha256Sum`も分離

* Fix typo

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

* target-dirも注入

* `build_envs`と`runtime_envs`もconst変数化

* Minor refactor

* `testcases`もconst変数化

* L1 normで比較し、`decode_forward`は一旦諦める

* `close_l1`を手作り

* コメントをアップデート

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

* `TestCase`をトレイトオブジェクトにする

* #400 に対応

* 分離

* `inventory::collect!`の場所を移動

* 余計な`//`を削除

* Minor refactor

* `TestSuite`という名前は最早適切ではないので改名

* 型引数名をリネーム

* 型引数名をリネーム

* indocを外す

* `CompatibleEngineLoadModelBeforeInitialize`を追加

* indocはもう使っていないため、`format_args_capture`を使う

* hexは使っていない

* `TestCase`と`Snapshots`をテストごとにファイル分離

* コメントの書き間違えを修正

* `_load_model_before_initialize`ではロガーが動いてるのが正しい

* `struct TestCase`で統一する

* Minor refactor

* `fn() -> Box<dyn TestCase>`を"submit"する

* Minor refactor

* `FnBoxTestCase` → `TestCaseSubmission`

* コメントを追加

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

* コメントを追加

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

* `mask` → `log_mask`

* docstringを追加

* コメントを更新

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

* コメントを更新

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

* コメントを更新

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

* trailing commaが残っていたのを削除

* `case!`にdocを書く

* `release`のDLLをテストする

---------

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.

2 participants