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

docs: #902 の注意書き #903

Merged
merged 2 commits into from
Dec 24, 2024
Merged

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Dec 22, 2024

内容

非同期APIの

  1. モジュール
  2. cpu_num_threads

において、 #902 についての注意書きを書く。

関連 Issue

その他

@qryxip qryxip requested review from Hiroshiba and removed request for Hiroshiba December 22, 2024 17:36
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.

#902 から0指定が良くないかもと思う根拠がわからず、迷子になりそうです!
#902 側に推察の理屈を書くか、「試用したところこういう挙動になったため」などと経験を書くかすると良さそう!
別のとこに説明があるなら #902 からその説明へのリンクを書くだけでも。

まあドキュメントに書くならちょっと整備しておきたいかな、くらいの気持ちです。

ちなみになんでなんでしたっけ…。忘れてしまった
…。

@qryxip
Copy link
Member Author

qryxip commented Dec 23, 2024

ちょっと文面を変えました: b72f826 (#903)

ちなみに「未調査だけど…かもしれない」という形になっているのは、 #545 的に単純に手間を惜しみたかったからです。なのでこれ以上記述するとしたら #902 に追記する形になるかなと。ただ"物理コアの数+1が適切かも"とした時点で少なくとも迷子にはならなさそう…?

@Hiroshiba
Copy link
Member

あ、ユーザー向けには現状の案内で良さそうに思いました!
開発者的になぜコア数+1が良いのか、whyがわからないって感じかな。

まあでもそれはこのPRに関係なく、 #902 の話かもです。
レビュー進めます!

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!!

ちゃんと「非同期版使う際はこの設定が良いかも」という雰囲気が伝わるようになってて良さそう!

@qryxip qryxip merged commit babb3b7 into VOICEVOX:main Dec 24, 2024
30 checks passed
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