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

APIが投げうるエラーについて網羅的にまとめてAPIドキュメントに書く #564

Open
3 tasks done
Tracked by #545
qryxip opened this issue Aug 2, 2023 · 3 comments
Open
3 tasks done
Tracked by #545

Comments

@qryxip
Copy link
Member

qryxip commented Aug 2, 2023

内容

各APIドキュメントに、APIが投げうるエラーについて網羅的にまとめます。

Pros 良くなる点

  • APIドキュメントとしてのわかりやすさの向上

Cons 悪くなる点

  • 手間

実現方法

調べ、書く。

VOICEVOXのバージョン

N/A

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

##532 でやるつもりでした。

@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 3, 2023

issue作成ありがとうございます!

これってC APIの話だと思うのですが、タイトルにそのように含めておくと終了条件が分かりやすいかなと思いました!
もし全てのAPIの話だとおそらく相当解決が困難なので、小分けにしておくとちょっとやりやすいかもです!

@qryxip
Copy link
Member Author

qryxip commented Aug 4, 2023

こちらもRust/C/Python全部に対してを意図していました。Rustdocの# ErrorsSphinxの:raises:を埋める形です。

段取りとしてはまずRust APIのエラーを全部調べて# Errorsにまとめ、それからC/Python APIのものを書くことを考えています。あまり多くないし、大丈夫なのではと思っています。

rg Error:: ./crates/voicevox_core -c
./crates/voicevox_core/src/voice_synthesizer.rs:1
./crates/voicevox_core/src/voice_model.rs:7
./crates/voicevox_core/src/devices.rs:1
./crates/voicevox_core/src/status.rs:12
./crates/voicevox_core/src/inference_core.rs:4
./crates/voicevox_core/src/user_dict/dict.rs:6
./crates/voicevox_core/src/user_dict/word.rs:9
./crates/voicevox_core/src/engine/open_jtalk.rs:12
./crates/voicevox_core/src/engine/full_context_label.rs:4

@Hiroshiba
Copy link
Member

なるほどです!
示されているのはエラーを発生させる関数の数でしょうか。便利。

今どの言語のエラーがすでにドキュメントになっているかかを他の人がすぐに確認できるようにしておくと、進捗度が分かって参加しやすかったりしていいのかなぁとちょっと思いました。
issueを分けておけば、誰かが取り組みたいと思った時に手をつけやすいかも・・・?
あるいはRust``C``Pythonと言語のリストだけ書いてタスクリストとして一番上で案内するとか・・・?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants