run.py: __main__
実装をmain
関数スコープに移動させる
#761
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
run.py
の__main__
実装を関数スコープにしたい #752の実装です。
run.py
のif __name__ == "__main__":
以下の実装を、新しく追加したmain
関数に移動させます。generate_app
関数内でグローバルスコープ変数(run.py
のファイルスコープ変数)を参照することを防ぎ、run.py
以外のテストなどからgenerate_app
が呼び出された際の不具合を防ぎます。関連 Issue
run.py
の__main__
実装を関数スコープにしたい #752スクリーンショット・動画など
その他
default_cors_policy_mode
変数の削除についてもともと未使用変数だったため削除しました。引数
--cors_policy_mode
が指定されなかった場合、リポジトリのdefault_setting.yml
に記述されたlocalapps
が使用されます。generate_app
関数内のenable_cancellable_synthesis
判定についてgenerate_app
関数にenable_cancellable_synthesis: bool
引数を追加する代わりに、cancellable_engine: CancellableEngine | None
引数を追加して、is not None
で判定するようにしました。Cancellable Engineを有効化するコマンド例(Windows)
poetry run python run.py --voicevox_dir "$HOME/AppData/Local/Programs/VOICEVOX" --enable_cancellable_synthesis --init_processes 2