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

manifest.jsonにブランド名追加 #567

Merged
merged 2 commits into from
Jan 9, 2023

Conversation

Hiroshiba
Copy link
Member

内容

エンジンの名前ではなく、「VOICEVOX」などの名前を表示したい機会があるので、ブランド名を書くためのキーを用意します。

関連 Issue

その他

ついでにnameをEngineに変えました。
いろいろ表記ゆれしますが、まあとりあえずこんな感じで行こうかなと!

  • リポジトリ名・開発コード名
    • VOICEVOX ENGINE
  • 日本語で言及するとき
    • VOICEVOX エンジン
  • name(かっこつけたいとき)
    • VOICEVOX Engine

@Hiroshiba Hiroshiba requested a review from a team as a code owner January 8, 2023 08:02
@Hiroshiba Hiroshiba requested review from y-chan and removed request for a team January 8, 2023 08:02
@Hiroshiba
Copy link
Member Author

@shirowanisan さんもレビュー頂けると助かります・・・!

@github-actions
Copy link

github-actions bot commented Jan 8, 2023

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 155 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 206 166 coverage-19%
voicevox_engine/synthesis_engine/make_synthesis_engines.py 57 49 coverage-14%
voicevox_engine/synthesis_engine/synthesis_engine.py 133 12 coverage-91%
voicevox_engine/synthesis_engine/synthesis_engine_base.py 67 9 coverage-87%
voicevox_engine/user_dict.py 129 6 coverage-95%
voicevox_engine/utility/init.py 3 0 coverage-100%
voicevox_engine/utility/connect_base64_waves.py 37 0 coverage-100%
voicevox_engine/utility/path_utility.py 26 6 coverage-77%
TOTAL 1238 262 coverage-79%

@shirowanisan
Copy link
Contributor

shirowanisan commented Jan 8, 2023

@Hiroshiba
LGTMです!

ただ、今はrelease-14.0に追従してみなさん作っておられると思うので、
各エンジンがbrand_nameがない状態でリリースされたものと混在する可能性があります。

UI側で「brand_nameがないもの」を読み込むときにエラーになってしまうと
ユーザが導入できないと混乱する可能性があるので、注意が必要ですね。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jan 8, 2023

ありがとうございます!

たしかにそうですね・・・!
release-0.14は最終的に色々変わっていくと思うので、最後に再度案内したいと思います 🙇‍♂️

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!
こちらの方でもこの変更の取り込みを行いたいと思います。

@Hiroshiba
Copy link
Member Author

マージします!

@Hiroshiba Hiroshiba merged commit 9de65e7 into master Jan 9, 2023
@Hiroshiba Hiroshiba deleted the manifest.jsonにブランド名追加 branch January 9, 2023 08:19
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.

3 participants