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

ダミーのspeaker_infoデータを追加する #337

Open
takana-v opened this issue Feb 20, 2022 · 6 comments · May be fixed by #1052
Open

ダミーのspeaker_infoデータを追加する #337

takana-v opened this issue Feb 20, 2022 · 6 comments · May be fixed by #1052
Labels
機能向上 状態:実装 実装をおこなっている状態 非アクティブ

Comments

@takana-v
Copy link
Member

内容

python run.py --voicevox_dir=/path/to/vve で起動し、エディタを起動するとエラーが発生します。
これは「雨晴はう」のuuidに紐づくダミーのspeaker_infoがまだ無いためです。
これを解消するためにダミーのspeaker_infoデータを追加します。

Pros 良くなる点

エラーの改善

Cons 悪くなる点

なし

実現方法

ダミーのspeaker_infoデータを追加する。

その他

キャラが追加されるたびにダミーのspeaker_infoデータを追加する必要があります。
これは正直面倒なので何かいい方法を考えたいです。

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 20, 2022

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

キャラが追加されるたびにダミーのspeaker_infoデータを追加する必要があります。
これは正直面倒なので何かいい方法を考えたいです。

たしかに・・・
良い方法かはちょっと自信がないのですが、speaker_infoもvoicevox_dir以下のディレクトリから読むようにすれば回避できたりしそうでしょうか。
(本当はspeaker_infoの情報もコアに移して、コアから情報を持ってくるようにするのが綺麗そうですが、時間がかかるので。。)

@takana-v
Copy link
Member Author

良い方法かはちょっと自信がないのですが、speaker_infoもvoicevox_dir以下のディレクトリから読むようにすれば回避できたりしそうでしょうか。

voicevox_dirの指定の場合はそれで大丈夫そうですが、コアを直接指定する場合はどうしようもありませんね...


こちらのIssueで提案されている、

UUIDとディレクトリ名のマッピングjsonを配置する

を実装し、開発版ではすべて同じディレクトリを指すようにすれば手間が減るかなと思います。
(全て同じアイコンになってしまいますが...)

@tarepan
Copy link
Contributor

tarepan commented Mar 5, 2024

キャラが追加されるたびにダミーのspeaker_infoデータを追加

現在の VIOCEVOX ENGINE は /speaker_info/ 以下に4人分の dummy speaker info をハードコードしていると認識しています。
製品版のキャラ増に関わらず dummy は一定であり、上記の記述とは違う状況のように見えます。

@Hiroshiba
こちらの issue は具体的にエディタのどの API call がバグ源となっているものでしょうか? あるいは解消済み?(当時のコンテキストがイマイチ分からず🙇)

@Hiroshiba
Copy link
Member

音声合成コアだけを直接参照してエンジン起動しつつエディタを起動すると、画像が見つからなくてエディタが起動しないという内容だと思います!

のコメントの内容とかなり近いということに今気づきました…。

@takana-v さん的に今は困ってなければcloseで良いのかなと思いました!
(まあ、画像がなければダミー画像を返すみたいなエンジン起動コマンド引数あると個人的には便利な気もちょっとします)

@takana-v
Copy link
Member Author

takana-v commented Mar 7, 2024

このIssueを建てた理由は、speaker_infoがエディタでは必須であるため、開発環境で製品版のコアやエンジンを参照して起動した時にエラーになるからだったと記憶しています。
(製品版コア実装話者のダミーデータが全て含まれていたため以前は無問題→製品版コアに「雨晴はう」が追加されて問題が発覚といった流れだったような気がします)

#1052 がマージされれば、こちらのIssueはCloseで問題ないと思います。

@tarepan tarepan self-assigned this Mar 7, 2024
@tarepan tarepan added 状態:実装 実装をおこなっている状態 and removed 要議論 実行する前に議論が必要そうなもの labels Mar 8, 2024
@tarepan tarepan removed their assignment Mar 8, 2024
@tarepan tarepan removed the 優先度:低 (運用中止) label Mar 10, 2024
Copy link

github-actions bot commented Sep 7, 2024

本 Issue は直近 180 日間で活動がありません。今後の方針について VOICEVOX チームによる再検討がおこなわれる予定です。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
機能向上 状態:実装 実装をおこなっている状態 非アクティブ
Projects
None yet
3 participants