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

マニフェストに能力を追加 #456

Merged
merged 1 commit into from
Sep 1, 2022
Merged

マニフェストに能力を追加 #456

merged 1 commit into from
Sep 1, 2022

Conversation

Hiroshiba
Copy link
Member

内容

エンジンマニフェストに、エンジンが備えている能力の一覧を付け足しました。
また、APIでその能力一覧を返すようにしました。

関連 Issue

close #362

@Hiroshiba Hiroshiba requested a review from takana-v August 27, 2022 13:43
@Hiroshiba
Copy link
Member Author

@takana-v さん、もしよければレビューよろしくお願いします・・・!

"supported_features": {
"adjust_mora_pitch": {
"type": "bool",
"value": true,
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issueでは「default」だったのですが、そもそもここをエディタ側などで変更することがないので、普通にvalueにしました。

"adjust_mora_pitch": {
"type": "bool",
"value": true,
"name": "モーラごとの音高の調整"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

別にユーザーに見せる場所ではないので、descriptionはなくても良いかなと思って省いてみました。

@github-actions
Copy link

Coverage Result

Resultを開く
Name Stmts Miss Cover
voicevox_engine/init.py 1 0 coverage-100%
voicevox_engine/acoustic_feature_extractor.py 75 0 coverage-100%
voicevox_engine/dev/synthesis_engine/init.py 2 0 coverage-100%
voicevox_engine/dev/synthesis_engine/mock.py 36 2 coverage-94%
voicevox_engine/full_context_label.py 162 3 coverage-98%
voicevox_engine/kana_parser.py 86 1 coverage-99%
voicevox_engine/model.py 154 7 coverage-95%
voicevox_engine/mora_list.py 4 0 coverage-100%
voicevox_engine/part_of_speech_data.py 5 0 coverage-100%
voicevox_engine/preset/Preset.py 12 0 coverage-100%
voicevox_engine/preset/PresetLoader.py 34 1 coverage-97%
voicevox_engine/preset/init.py 3 0 coverage-100%
voicevox_engine/synthesis_engine/init.py 5 0 coverage-100%
voicevox_engine/synthesis_engine/core_wrapper.py 199 159 coverage-20%
voicevox_engine/synthesis_engine/make_synthesis_engines.py 48 39 coverage-19%
voicevox_engine/synthesis_engine/synthesis_engine.py 133 12 coverage-91%
voicevox_engine/synthesis_engine/synthesis_engine_base.py 66 9 coverage-86%
voicevox_engine/user_dict.py 131 10 coverage-92%
voicevox_engine/utility/init.py 3 0 coverage-100%
voicevox_engine/utility/connect_base64_waves.py 35 3 coverage-91%
voicevox_engine/utility/engine_root.py 9 2 coverage-78%
TOTAL 1203 248 coverage-79%

Copy link
Member

@takana-v takana-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member Author

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tantan-tanuki さん

UI作成時に課題になりそうな点などがあるか、軽くレビュー頂いても良いでしょうか 🙇‍♂️

Copy link
Contributor

@tantan-tanuki tantan-tanuki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

他の説明文が全て日本語なので合わせるなら「モーラ」という単語は「拍」のほうが良いのかなと感じたのですが
そこまで杓子定規にして逆に解りづらくなるのは本末転倒になりますし
私がそっち方面の知識に疎いため

「モーラごとの音高の調整」
「拍ごとの音高の調整」

のどちらの文言がVOICEVOXユーザにとってわかりやすいのかという判断がつかなかったためコメントとさせて頂きました

その他全体の修正内容的には問題はないと思いますのでモーラのままで問題ないという判断をいただければApproveにしようと思います

@Hiroshiba
Copy link
Member Author

レビューありがとうございます!

確かにここだけ横文字なので若干不自然な気がしました。
「拍」は、音楽のリズムの方の「拍」と混同されそうなので、あえてモーラの方を使っていました。
(本当はもっとわかりやすい日本語を使いたいのですが、適切なものがなく。。)

そういう意味ではモーラのままでも問題なのですが、まあ、モーラで行きましょうということで。。。 🙇

@y-chan
Copy link
Member

y-chan commented Sep 1, 2022

もうすでにHiroshibaさんの決定が出ていますが、過去にもモーラの表現・表記に関して議論があったので載せておきます。
#44 (comment)

Copy link
Contributor

@tantan-tanuki tantan-tanuki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

過去に話題に上がっていたのですね失礼しました
それでは承認します!

多分私がモーラという単語に引っかかったのは子供の頃こいつで遊んでたせいもあると思います😅

@Hiroshiba
Copy link
Member Author

ありがとうございます!マージします!

@Hiroshiba Hiroshiba merged commit 33c4d1e into VOICEVOX:master Sep 1, 2022
@Hiroshiba Hiroshiba deleted the マニフェストに能力を追加 branch September 1, 2022 04:33
sabonerune pushed a commit to sabonerune/voicevox_engine that referenced this pull request Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

エンジンの機能リストAPIの検討
4 participants