SynthesisEngineの抽象基底クラスSynthesisEngineBaseを追加 #227

aoirint · 2021-12-09T20:42:36Z

内容

#199 にて、run.py の複雑さを減らすため、
create_accent_phrases 関数を run.py から SynthesisEngine クラスに移動させましたが、
Mockに create_accent_phrases 関数を実装し忘れていたため、Mockが動作しなくなっていました。

このPRは、SynthesisEngineとMockの両方が持つべき処理を SynthesisEngineBase クラスに実装し、
SynthesisEngineとMockは SynthesisEngineBase クラスを継承するようにして、
Mockが動作するようにします。

また、Mockの動作を確認するため、MockSynthesisEngineのテストを追加します。

SynthesisEngineBaseでは、yukarin_s_forwarder、yukarin_sa_forwarder、decode_forwarder、speakersに依存する処理は未実装とし、呼び出し時に例外を投げます（が、abc.abstractmethodを指定しているので、あくまでマーカー＝実際に使われることはない、と思います）。

https://docs.python.org/ja/3.8/library/exceptions.html#NotImplementedError

SynthesisEngineBaseは抽象基底クラスとし、抽象メソッドが実装されていない場合にインスタンスを作成できないようにしています（abc.abstractmethod）。

https://docs.python.org/ja/3.8/library/abc.html#abc.abstractmethod

その他

命名規則ちょっと悩みます。

SynthesisEngineBase、MockSynthesisEngine
- 補完がしやすそうなので採用しています
BaseSynthesisEngine、MockSynthesisEngine
- 統一感があります
SynthesisEngineBase、SynthesisEngineMock
- こちらでもいいかも
BaseSynthesisEngine、SynthesisEngineMock

github-actions · 2021-12-09T20:45:28Z

Coverage Result

Resultを開く

Name	Stmts	Miss
voicevox_engine/init.py	0	0
voicevox_engine/acoustic_feature_extractor.py	85	0
voicevox_engine/dev/synthesis_engine/init.py	2	0
voicevox_engine/dev/synthesis_engine/mock.py	30	1
voicevox_engine/full_context_label.py	160	7
voicevox_engine/kana_parser.py	75	1
voicevox_engine/model.py	70	7
voicevox_engine/mora_list.py	4	0
voicevox_engine/preset/Preset.py	12	0
voicevox_engine/preset/PresetLoader.py	34	1
voicevox_engine/preset/init.py	3	0
voicevox_engine/synthesis_engine/init.py	5	0
voicevox_engine/synthesis_engine/forwarder.py	76	66
voicevox_engine/synthesis_engine/make_synthesis_engine.py	23	18
voicevox_engine/synthesis_engine/synthesis_engine.py	108	0
voicevox_engine/synthesis_engine/synthesis_engine_base.py	30	10
voicevox_engine/utility/init.py	2	0
voicevox_engine/utility/connect_base64_waves.py	35	3
TOTAL	754	114