Skip to content
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

C APIのエラー表示をtracing::error!にする #600

Merged
merged 2 commits into from
Sep 5, 2023

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Sep 5, 2023

内容

C APIのエラーの表示をtracing::error!のものにします。

表示内容についてですが、デファクトな<anyhow::Error as Debug>の表示を 一行に折り畳む 真似た形にしました。

Error: Failed to read instrs from ./path/to/instrs.json

Caused by:
    No such file or directory (os error 2)

今のエラーの設計だとちょっと見れたものではなくなっていますが、これをどうにかするのは別PRにします。

2023-09-06T01:04:59.020162+09:00 ERROR voicevox_core::helpers: 入力テキストをAquesTalk風記法としてパースすることに失敗しました,Parse Error: second accent cannot be set at an accent phrase: ア''
2023-09-06T01:04:59.020201+09:00 ERROR voicevox_core::helpers: Caused by: 入力テキストをAquesTalk風記法としてパースすることに失敗しました,Parse Error: second accent cannot be set at an accent phrase: ア''
2023-09-06T01:04:59.020207+09:00 ERROR voicevox_core::helpers: Caused by: Parse Error: second accent cannot be set at an accent phrase: ア''
2023-09-06T01:03:24.955815+09:00 ERROR voicevox_core::helpers: `/home/ryo/src/github.com/VOICEVOX/voicevox_core/example/cpp/unix/./voicevox_core/model/a.vvm`の読み込みに失敗しました: ZIPファイルとして開くことができませんでした: an upstream reader returned an error: Is a directory (os error 21)
2023-09-06T01:03:24.955851+09:00 ERROR voicevox_core::helpers: Caused by: `/home/ryo/src/github.com/VOICEVOX/voicevox_core/example/cpp/unix/./voicevox_core/model/a.vvm`の読み込みに失敗しました: ZIPファイルとして開くことができませんでした: an upstream reader returned an error: Is a directory (os error 21)
2023-09-06T01:03:24.955855+09:00 ERROR voicevox_core::helpers: Caused by: an upstream reader returned an error: Is a directory (os error 21)
2023-09-06T01:03:24.955859+09:00 ERROR voicevox_core::helpers: Caused by: Is a directory (os error 21)

関連 Issue

#580, #589 の続きです。

ref #545

その他

Backtraceの表示ができればなお良いのですが、残念ながら今のRustだと... (eyreに切り替えるならその限りではないですが)

@qryxip qryxip mentioned this pull request Sep 5, 2023
69 tasks
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!!

一塊にしてログを出せるとかっこよさそうですね!

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