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

キャラクターリソースをエンジンに含め、preview版ビルドを作る #533

Closed
10 tasks done
Hiroshiba opened this issue Nov 29, 2021 · 9 comments
Closed
10 tasks done

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Nov 29, 2021

内容

CharacterInfoをエンジンから取って来るようにする予定です。

今の製品版エンジン(0.9.3)はちょっとバグっていて、speakers_infoのデータが正しく取れていません。
なので↑のプルリクエストをマージすると、製品版エンジンを使ったエディタの開発できなくなってしまいます。

ということで、0.10のエンジンのプレビュー版ビルドを作成します。
ちょっと複雑な手順が必要になってしまうので、下記にまとめました。

実現方法

VOICEVOXのバージョン

0.9.3

その他

speakers_infoのデータが正しく取れていないのに気づけませんでした、すみません。。

@takana-v
Copy link
Member

現在の状態ではresourceの画像ファイルの名前がmockと異なっており、正常に画像を取得できません。
resourceの方を変えてしまうとエディタのビルドの方に影響が出てしまうので、エンジン側で対応したいと思います。
(オプショナル)エンジンのリソースモックにキャラクターを2人追加と合わせて行いたいと思います。
ref VOICEVOX/voicevox_engine#206

@Hiroshiba
Copy link
Member Author

なるほど!!
エンジン側の方を合わせるという手がありますね!!
ぜひお願いしたいです!

@Hiroshiba
Copy link
Member Author

エンジンのビルドを開始しました!
https://github.com/VOICEVOX/voicevox_engine/releases/tag/0.10.preview.1

@takana-v
Copy link
Member

takana-v commented Dec 1, 2021

0.10.preview.1をチェックしたところ、speaker_info内のフォルダ名がuuidではありませんでした。
actionsを見るとCheckoutがスキップされていたのでキャッシュを使ってしまっているのかもしれません。

追記:
おそらくエンジンのbuild.ymlのmatrix.voicevox_resource_versionが古いままなのが原因な気がします。
ブランチが無いのでPRをどこに出せばいいか分かりませんが...

@shirowanisan
Copy link
Contributor

shirowanisan commented Dec 1, 2021

横から失礼します。
0.10.preview.1でengine側のdummy1とdummy2のスタイルの順番違いが直っていないように思います。

dummy1
0.pngのスタイルが1_2 → 正しくは1_1
https://github.com/VOICEVOX/voicevox_engine/blob/master/speaker_info/7ffcb7ce-00ec-4bdc-82cd-45a8889e43ff/icons/0.png
2.pngのスタイルが1_1 → 正しくは1_2
https://github.com/VOICEVOX/voicevox_engine/blob/master/speaker_info/7ffcb7ce-00ec-4bdc-82cd-45a8889e43ff/icons/2.png

0_001-003のサンプル音声が「あまあま」 → 正しくは「ノーマル」
2_001-003のサンプル音声が「ノーマル」 → 正しくは「あまあま」

dummy2
1.pngのスタイルが2_2 → 正しくは2_1
https://github.com/VOICEVOX/voicevox_engine/blob/master/speaker_info/388f246b-8c41-4ac1-8e2d-5d79f3ff56d9/icons/1.png
3.pngのスタイルが2_1 → 正しくは2_2
https://github.com/VOICEVOX/voicevox_engine/blob/master/speaker_info/7ffcb7ce-00ec-4bdc-82cd-45a8889e43ff/icons/2.png

1_001-003のサンプル音声が「あまあま」 → 正しくは「ノーマル」
3_001-003のサンプル音声が「ノーマル」 → 正しくは「あまあま」

@y-chan
Copy link
Member

y-chan commented Dec 1, 2021

@shirowanisan さん
現状で問題ないと思います。

というのも、VOICEVOXは公開当初、 @Hiroshiba さんの手違いによって「あまあま」スタイルの音声が提供されていた経緯があります。
https://twitter.com/hiho_karuta/status/1447133506161688578?t=t1BMLwwxI7UzY8MRNqLUaA&s=19

現状のVOICEVOXでは、過去バージョンとのプロジェクトファイルの互換性を保つために、speaker idの0と1の音声を「あまあま」スタイルのまま、変えずに保持しています。

なので、間違っている訳ではなく、意図して順番を変更しているものと思います。

@shirowanisan
Copy link
Contributor

@y-chan おお、そうだったんですね。
了解です。仕様であるならば問題ありません。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Dec 1, 2021

0.10.preview.1をチェックしたところ、speaker_info内のフォルダ名がuuidではありませんでした。 actionsを見るとCheckoutがスキップされていたのでキャッシュを使ってしまっているのかもしれません。

追記: おそらくエンジンのbuild.ymlのmatrix.voicevox_resource_versionが古いままなのが原因な気がします。 ブランチが無いのでPRをどこに出せばいいか分かりませんが...

おっと、ほんとですね・・・ ご指摘感謝です!
release-0.10のbranchを作って修正してrelease作り直してみます。

@Hiroshiba
Copy link
Member Author

周知用issueも作ったので、このissueはcloseします!

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

4 participants