追加: SynthesisEngine Core情報都度取得

[tarepan]

内容

「SynthesisEngine Core情報都度取得」機能の追加

関連 Issue

resolve #802

[github-actions]

Coverage Result

Resultを開く

Name	Stmts	Miss	Cover
run.py	480	326
voicevox_engine/init.py	1	0
voicevox_engine/acoustic_feature_extractor.py	23	0
voicevox_engine/cancellable_engine.py	91	71
voicevox_engine/dev/core/init.py	2	0
voicevox_engine/dev/core/mock.py	27	12
voicevox_engine/dev/synthesis_engine/init.py	2	0
voicevox_engine/dev/synthesis_engine/mock.py	36	2
voicevox_engine/engine_manifest/EngineManifest.py	34	0
voicevox_engine/engine_manifest/EngineManifestLoader.py	12	0
voicevox_engine/engine_manifest/init.py	3	0
voicevox_engine/full_context_label.py	162	3
voicevox_engine/kana_parser.py	86	1
voicevox_engine/library_manager.py	93	5
voicevox_engine/metas/Metas.py	33	0
voicevox_engine/metas/MetasStore.py	29	12
voicevox_engine/metas/init.py	2	0
voicevox_engine/model.py	162	9
voicevox_engine/mora_list.py	4	0
voicevox_engine/morphing.py	70	46
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/setting/Setting.py	11	0
voicevox_engine/setting/SettingLoader.py	19	0
voicevox_engine/setting/init.py	3	0
voicevox_engine/synthesis_engine/init.py	5	0
voicevox_engine/synthesis_engine/core_wrapper.py	201	146
voicevox_engine/synthesis_engine/make_synthesis_engines.py	59	30
voicevox_engine/synthesis_engine/synthesis_engine.py	128	13
voicevox_engine/synthesis_engine/synthesis_engine_base.py	67	9
voicevox_engine/user_dict.py	144	12
voicevox_engine/utility/init.py	5	0
voicevox_engine/utility/connect_base64_waves.py	37	0
voicevox_engine/utility/core_version_utility.py	8	1
voicevox_engine/utility/mutex_utility.py	10	0
voicevox_engine/utility/path_utility.py	26	8
TOTAL	2179	708

[Hiroshiba]

なるほどです、キャッシュせず毎回ロードする形ですね！
確かにloadedとかはコア側での管理なのに、話者情報などはエンジン側で管理しているのは不揃いに感じました。
毎回取得・dictに変更するのは計算コストが少し増しますが、全然許容範囲に思います。

コア実装に詳しい @takana-v さんや @PickledChair さん的になにか問題になりそうな点に気づいたらお聞きしたいです！

[takana-v]

Synthesisエンジン内で話者情報を保持していたのは、昔のコードの名残かと思われます。

• モデルの読み込みを起動時から変更 #400

これより以前は、SynthesisEngineクラスを作る時に引数として話者情報を渡していました。

ということで、問題ないかと思います。

[PickledChair]

特に問題はないと思いました……！　SynthesisEngine がコアの情報をキャッシュしていた理由については takana さんのおっしゃる通りだと思います。また、Issue で tarepan さんが触れている通り、性能低下の懸念はあまりないと思います。適切なリファクタリングだと感じました

[Hiroshiba]

お二人ともありがとうございます！！
マージさせていただこうと思います！