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

整理: 貢献者ガイド節への集約 #838

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented Dec 9, 2023

内容

README.md コンテンツのうち、コントリビュータのみに関連する内容を新設の「貢献者ガイド」節へ集約してリファクタリング

コンテンツ変更一切無し、集約のみ

関連 Issue

part of #687

Copy link

github-actions bot commented Dec 9, 2023

Coverage Result

Resultを開く
Name Stmts Miss Cover
run.py 481 327 coverage-32%
voicevox_engine/init.py 1 0 coverage-100%
voicevox_engine/acoustic_feature_extractor.py 19 1 coverage-95%
voicevox_engine/cancellable_engine.py 91 71 coverage-22%
voicevox_engine/dev/core/init.py 2 0 coverage-100%
voicevox_engine/dev/core/mock.py 27 12 coverage-56%
voicevox_engine/dev/synthesis_engine/init.py 2 0 coverage-100%
voicevox_engine/dev/synthesis_engine/mock.py 38 2 coverage-95%
voicevox_engine/engine_manifest/EngineManifest.py 34 0 coverage-100%
voicevox_engine/engine_manifest/EngineManifestLoader.py 12 0 coverage-100%
voicevox_engine/engine_manifest/init.py 3 0 coverage-100%
voicevox_engine/full_context_label.py 162 3 coverage-98%
voicevox_engine/kana_parser.py 86 1 coverage-99%
voicevox_engine/library_manager.py 93 5 coverage-95%
voicevox_engine/metas/Metas.py 33 0 coverage-100%
voicevox_engine/metas/MetasStore.py 29 12 coverage-59%
voicevox_engine/metas/init.py 2 0 coverage-100%
voicevox_engine/model.py 162 9 coverage-94%
voicevox_engine/mora_list.py 4 0 coverage-100%
voicevox_engine/morphing.py 70 46 coverage-34%
voicevox_engine/part_of_speech_data.py 5 0 coverage-100%
voicevox_engine/preset/Preset.py 12 0 coverage-100%
voicevox_engine/preset/PresetError.py 2 0 coverage-100%
voicevox_engine/preset/PresetManager.py 81 2 coverage-98%
voicevox_engine/preset/init.py 4 0 coverage-100%
voicevox_engine/setting/Setting.py 11 0 coverage-100%
voicevox_engine/setting/SettingLoader.py 19 0 coverage-100%
voicevox_engine/setting/init.py 3 0 coverage-100%
voicevox_engine/synthesis_engine/init.py 5 0 coverage-100%
voicevox_engine/synthesis_engine/core_wrapper.py 202 147 coverage-27%
voicevox_engine/synthesis_engine/make_synthesis_engines.py 59 30 coverage-49%
voicevox_engine/synthesis_engine/synthesis_engine.py 172 13 coverage-92%
voicevox_engine/synthesis_engine/synthesis_engine_base.py 71 10 coverage-86%
voicevox_engine/user_dict.py 144 12 coverage-92%
voicevox_engine/utility/init.py 5 0 coverage-100%
voicevox_engine/utility/connect_base64_waves.py 37 0 coverage-100%
voicevox_engine/utility/core_version_utility.py 8 1 coverage-88%
voicevox_engine/utility/mutex_utility.py 10 0 coverage-100%
voicevox_engine/utility/path_utility.py 26 8 coverage-69%
TOTAL 2227 712 coverage-68%

@Hiroshiba
Copy link
Member

並び替え良いですね!!!

貢献者向けの情報って、実行やビルドの方法も書いていたほうが嬉しかったりしそうでしょうか?
なんとなくVueJSのガイドライン見た感じそんななのかな〜と!!
https://github.com/vuejs/core/blob/main/.github/contributing.md

@tarepan
Copy link
Contributor Author

tarepan commented Dec 9, 2023

実行やビルドの方法も書いていたほうが

はい、ビルド・実行の情報も(初回コントリビュート時に)欲しい情報でした。

しかし、これらの情報を貢献者ガイドへ組み入れると「開発者に伝わらない恐れ」というデメリットが出てきます。

voicevox_engine レポジトリには主に3通りの利用者・用途があります:

  • User ソフトウェアユーザー: ビルド済み VOICEVOX ENGINE ソフトウェアの利用ガイド
  • Dev 開発者: VOICEVOX マルチエンジン開発出発点
  • Contrib コントリビューター: 改良対象

ビルド・実行の手順情報は User に不要な一方、DevContrib には必須です。
そして Dev にとって「voicevox_engine への貢献方法」は知る必要の無い情報です。
よって Dev は貢献者ガイドを(とりあえず)読み飛ばす可能性が高いです。

ゆえにビルド・実行の手順を貢献者ガイドへ入れてしまうと、Dev が必要な情報にたどり着けないリスクを上げてしまいます。

コントリビュータはREADMEを一通り読むと考えると、貢献者ガイド節より前にビルド・実行の手順が書いてあれば最低限情報は伝わる気がします。
PR作成時にREADMEをスクロールして「どこだったっけ…?」とするのは確かに不便なので、貢献者ガイド内にビルド・実行節へのリンクを貼るのは手かもしれません。

@Hiroshiba
Copy link
Member

@tarepan なるほどです!
VOICEVOXの人数バランス的に、マルチエンジンを作る人よりもコントリビューターの方が圧倒的に多いので、そちらにUXを合わせるのが良いのかなと思いました!
ただまあコントリビューター・マルチエンジン開発者どちらにも必要な情報というのはあると思うので、そこの案内をどうするかは確かに考える必要があるかなと思いました。

コントリビューターのところに実行や環境構築やビルドの方法を書いちゃって、マルチエンジン実装方法を紹介するとこに「VOICEVOXエンジンの実行方法は貢献者ガイドラインの~~にあります」と案内するのはどうでしょう?

@tarepan
Copy link
Contributor Author

tarepan commented Dec 10, 2023

VOICEVOXの人数バランス ... コントリビューターの方が圧倒的に多い

納得です。

マルチエンジン実装方法を紹介するとこに「VOICEVOXエンジンの実行方法は貢献者ガイドラインの~~にあります」と案内

👍
こちらの方針で draft PR を再整理し、正式PR化します。

@tarepan tarepan marked this pull request as ready for review December 10, 2023 14:37
@tarepan tarepan requested a review from a team as a code owner December 10, 2023 14:37
@tarepan tarepan requested review from Hiroshiba and removed request for a team December 10, 2023 14:37
Copy link
Member

@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.

LGTM!!!

挑戦ありがとうございました!!

README.md Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit 3e6a739 into VOICEVOX:master Dec 12, 2023
3 checks passed
@tarepan tarepan deleted the refactor/contrib branch December 12, 2023 19:25
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.

2 participants