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

default_setting.yml を Dockerfile に追加する #562

Merged
merged 2 commits into from
Jan 6, 2023

Conversation

smly
Copy link
Contributor

@smly smly commented Jan 6, 2023

内容

この PR は default_setting.yml を Docker コンテナに含めるように変更することで、Docker コンテナが実行できない問題を解消する。

先週マージされた差分 cfc5ff1 にて default_setting.yml が追加されています。この差分にある SettingLoader.py はデフォルトの設定ファイルのパスにアクセスしようとします。Docker コンテナ内では /opt/voicevox_engine/default_setting.yml となりますが、これは Dockerfile で追加されていません。また Dockerfile で記述されている run.py の引数でも設定ファイルは指定されていません。そのため README に記載されている手順で Docker 版を pull & run して実行しようとするとエラーになります。

エラーメッセージは以下の通りです:

$ docker pull voicevox/voicevox_engine:cpu-ubuntu20.04-latest
$ docker run --rm -it -p '127.0.0.1:50021:50021' voicevox/voicevox_engine:cpu-ubuntu20.04-latest
+ cat /opt/voicevox_engine/README.md
## 許諾内容
(snip)
## 免責事項

本ソフトウェアにより生じた損害・不利益について、製作者は一切の責任を負いません。

## その他

ご利用の際は VOICEVOX を利用したことがわかるクレジット表記が必要です。
+ exec gosu user /opt/python/bin/python3 ./run.py --voicelib_dir /opt/voicevox_core/ --runtime_dir /opt/onnxruntime/lib --host 0.0.0.0
Warning: cpu_num_threads is set to 0. ( The library leaves the decision to the synthesis runtime )
Traceback (most recent call last):
  File "./run.py", line 1060, in <module>
    settings = setting_loader.load_setting_file()
  File "/opt/voicevox_engine/voicevox_engine/setting/SettingLoader.py", line 18, in load_setting_file
    setting = yaml.safe_load(DEFAULT_SETTING_PATH.read_text(encoding="utf-8"))
  File "/opt/python/lib/python3.8/pathlib.py", line 1236, in read_text
    with self.open(mode='r', encoding=encoding, errors=errors) as f:
  File "/opt/python/lib/python3.8/pathlib.py", line 1222, in open
    return io.open(self, mode, buffering, encoding, errors, newline,
  File "/opt/python/lib/python3.8/pathlib.py", line 1078, in _opener
    return self._accessor.open(self, flags, mode)
FileNotFoundError: [Errno 2] No such file or directory: '/opt/voicevox_engine/default_setting.yml'

関連 Issue

なし

スクリーンショット・動画など

なし

その他

@smly smly requested a review from a team as a code owner January 6, 2023 20:04
@smly smly requested review from y-chan and removed request for a team January 6, 2023 20:04
@github-actions
Copy link

github-actions bot commented Jan 6, 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%

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

PRありがとうございます!!
ビルド後にちゃんと動くかの実行テストするCIがあると良いかもですね。

(メンテナ側の共有です)
@y-chan さんすみません、レビュワーにアサインされてたのy-chanさんなんですがレビュー後に気づいたので進めちゃいました 🙇‍♂️

Dockerfile Show resolved Hide resolved
@smly
Copy link
Contributor Author

smly commented Jan 6, 2023

なるほど見落としてました。指摘ありがとうございます!差分追加しました

@smly smly closed this Jan 6, 2023
@smly smly reopened this Jan 6, 2023
@smly
Copy link
Contributor Author

smly commented Jan 6, 2023

すみませんボタン押し間違えまて Close しちゃいました。Reopen しました。

@Hiroshiba
Copy link
Member

問題ないと思うのでマージします!

@Hiroshiba Hiroshiba merged commit d824599 into VOICEVOX:master Jan 6, 2023
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