FIX: update_dictのエラーハンドリングを改善

[sabonerune]

内容

update_dictで一時ディレクトリから移動するエラーハンドリングが間違っていたので修正しました

その他

主にWindows環境でエンジンを多重起動したときに発生します。
この修正により辞書の更新に失敗してもそのまま起動することができます。
ただし、2つのエンジンが1つの辞書をロックすることになるので起動しているエンジンが1つになるまで辞書の更新ができない問題は解決しませんが…

[github-actions]

Coverage Result

Resultを開く

Name	Stmts	Miss	Cover
voicevox_engine/init.py	1	0
voicevox_engine/acoustic_feature_extractor.py	75	0
voicevox_engine/dev/synthesis_engine/init.py	2	0
voicevox_engine/dev/synthesis_engine/mock.py	36	2
voicevox_engine/full_context_label.py	162	3
voicevox_engine/kana_parser.py	86	1
voicevox_engine/metas/Metas.py	33	0
voicevox_engine/metas/MetasStore.py	29	14
voicevox_engine/metas/init.py	2	0
voicevox_engine/model.py	145	9
voicevox_engine/mora_list.py	4	0
voicevox_engine/part_of_speech_data.py	5	0
voicevox_engine/preset/Preset.py	12	0
voicevox_engine/preset/PresetError.py	2	0
voicevox_engine/preset/PresetManager.py	81	2
voicevox_engine/preset/init.py	4	0
voicevox_engine/synthesis_engine/init.py	5	0
voicevox_engine/synthesis_engine/core_wrapper.py	201	159
voicevox_engine/synthesis_engine/make_synthesis_engines.py	57	49
voicevox_engine/synthesis_engine/synthesis_engine.py	130	11
voicevox_engine/synthesis_engine/synthesis_engine_base.py	67	9
voicevox_engine/user_dict.py	134	10
voicevox_engine/utility/init.py	3	0
voicevox_engine/utility/connect_base64_waves.py	37	0
voicevox_engine/utility/path_utility.py	26	6
TOTAL	1339	275

[y-chan]

LGTM!

[Hiroshiba]

LGTM！！

[Hiroshiba]

意図がわからなくなっていて、Discordに議論が残っていたのでこちらに移します！
https://discord.com/channels/879570910208733277/893889888208977960/1073515005951090758

---

ヒホ — 2023/02/10 17:05

エディタでエンジンを素早く２回起動すると辞書がぶっ壊れた？っぽい報告が。
https://twitter.com/AreaMVP/status/1623925945324412929

---

サボ寝ルネ — 2023/02/10 17:51

過去にissueが上がってPython側の問題で修正が困難ということで未解決のままcloseしています。
#419

GitHub
OSError: [WinError 1] 書き込み権限が無いときにresolveがエラーになる · Issue #419 · VOICEVOX/voicevox_engine

---

サボ寝ルネ — 2023/02/10 18:00

そもそも起動時に毎回コンパイルするから%TEMP%ディレクトリで作成して移動する意味がないのでは?

---

t... — 2023/02/10 18:05

create_user_dictで失敗した時に辞書ファイルが消失しないように...だった記憶がありますね
（昔は起動時にコンパイルする仕様ではなかった）
strict=False にするのはファイルが存在しない場合にpyopenjtalkがエンジンごと殺してしまうので避けたい所

---

サボ寝ルネ — 2023/02/10 18:31

%TEMP%ではなくエンジンの設定ディレクトリ内に一時ファイル用のディレクトリ作ってしまうとか?
流石にここをRAMディスクにする人はいないだろうし

---

t... — 2023/02/10 19:33

それもありかもですね
dirオプションで設定できそう
https://docs.python.org/ja/3/library/tempfile.html#tempfile.TemporaryFile

---

サボ寝ルネ — 2023/02/10 20:12

(あとTEMPから辞書移動するときのエラーハンドリング間違っていることに気づいた)

---

ヒホ — 2023/02/11 00:47

たまに hoge.dic.tmp みたいに、後ろに.tmpつけて同じディレクトリに保存するとかもあるなと思いました。
VOICEVOX ENGINEがこんな感じの構成だった気がします。

---

ヒホ — 2023/02/11 11:57

(あとTEMPから辞書移動するときのエラーハンドリング間違っていることに気づいた)

これってどこでしょう 👀
（issue作ろうかなと思ったけど該当箇所がわからず）
わかりやすいよう user_dict.py へのリンク
https://github.com/VOICEVOX/voicevox_engine/blob/da60a0cc43cdc48759d53a7b8f5b4006d33a89df/voicevox_engine/user_dict.py

---

ヒホ — 2023/02/11 21:53

↑直接PRもらいました、ありです！！！ 🙏
#620