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

OSS版で製品版のvvmを読んだりしたときに気づけるエラーメッセージにする? #722

Open
Hiroshiba opened this issue Jan 4, 2024 · 0 comments · May be fixed by #825
Labels
機能向上 要議論 実行する前に議論が必要そうなもの

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Jan 4, 2024

内容

OSS版で製品版のvvmを読もうとすると、モデルデータを読むことができませんでしたになります。
sample.vvmのREADMEにはその案内がありますが、製品版を読もうとする場合にはこのREADMEを通らないので気づきづらいです。

これは結構起こりうるミスでわりとよく見かけるので、エラーメッセージの中に専用の注意書きを書いてもいいのかなと思いました。
例えばエラーメッセージを「モデルデータを読むことができませんでした。OSS版で製品版のモデルデータを読もうとしていませんか?」という風にするのはどうでしょうか。

Pros 良くなる点

エラーメッセージから原因を特定しやすくなる

Cons 悪くなる点

エラーがここに以外の箇所で起こることがあり得る・・・・・?(なさそうだけど)

実現方法

エラーメッセージを書き足す。

その他

まあ本来なら製品版かどうか、というかブランドの判定をつけるのが良さそうな気がしますが、とりあえず提案かねてissueを作ってみました!

@Hiroshiba Hiroshiba added 機能向上 要議論 実行する前に議論が必要そうなもの labels Jan 4, 2024
qryxip added a commit to qryxip/ort that referenced this issue Aug 31, 2024
qryxip added a commit to qryxip/ort that referenced this issue Aug 31, 2024
Hiroshiba pushed a commit to VOICEVOX/ort that referenced this issue Sep 1, 2024
* change: liberate VOICEVOX CORE

* `session.use_vv_bin`

* VOICEVOX/voicevox_core#722 用エラー
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
機能向上 要議論 実行する前に議論が必要そうなもの
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant