We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Python APIにおける例外 (e.g. InferenceFailedError)を、pyo3::create_exception!で宣言するのではなくPythonコード側に書きます。
InferenceFailedError
pyo3::create_exception!
N/A
ここら辺は文字列指定(py.import("voicevox_core")?.import("…")?)することになるのでtypoを考え、全例外をraiseしてexceptするだけのテストを加えた方がいいかもしれません。 (Java APIも今そうなっているので、Java APIと同時でもいいかも)
py.import("voicevox_core")?.import("…")?
raise
except
voicevox_core/crates/voicevox_core_python_api/src/convert.rs
Lines 190 to 209 in fa630ce
The text was updated successfully, but these errors were encountered:
No branches or pull requests
内容
Python APIにおける例外 (e.g.
InferenceFailedError
)を、pyo3::create_exception!
で宣言するのではなくPythonコード側に書きます。Pros 良くなる点
Cons 悪くなる点
実現方法
VOICEVOXのバージョン
N/A
OSの種類/ディストリ/バージョン
その他
ここら辺は文字列指定(
py.import("voicevox_core")?.import("…")?
)することになるのでtypoを考え、全例外をraise
してexcept
するだけのテストを加えた方がいいかもしれません。(Java APIも今そうなっているので、Java APIと同時でもいいかも)
voicevox_core/crates/voicevox_core_python_api/src/convert.rs
Lines 190 to 209 in fa630ce
The text was updated successfully, but these errors were encountered: