追加: TTSEngine スナップショット

[tarepan]

内容

TTSEngine 各メソッドのスナップショットを追加

関連 Issue

ref #978
ref #899

[tarepan]

test / test (ubuntu-20.04, 3.11.3) (pull_request) Successful in 1m
test / test (windows-latest, 3.11.3) (pull_request) Failing after 2m

=========================== short test summary info ===========================
FAILED test/tts_pipeline/test_tts_engine.py::test_mocked_synthesize_wave_output - assert [- snapshot] == [+ received]
    ......
      ...
        [
  -       1.30066,
  +       1.30067,
  -       1.30066
  +       1.30067
        ],
      ...
    ......
================== 1 failed, 131 passed, 5 warnings in 4.33s ==================

スナップショット値が Linux と Windows で一致しない...?

@Hiroshiba
数値計算周りで Linux/Windows 環境依存が問題になるケースって ENGINE で既出だったりしますか？

[Hiroshiba]

@tarepan

数値計算周りで Linux/Windows 環境依存が問題になるケースって ENGINE で既出だったりしますか？

onnxruntimeを使っている場合割と OS 依存で値が変わることはあったのですが、見たところモックを使っているので初めての現象な気がします！！！
こんなことありえるんですね！！！

問題はPythonかnumpyのどこかがfloat32だったりfloat64だったりすることな気がしています。
Pythonのfloatはデフォルトがdoubleで、numpyもdoubleなはずです。でもこれは一応実行環境のアーキテクチャーに依存しています。
もしかしたらGithub Actionsのubuntu20.04はfloat32で、windowsはflota64なのかもですが、そんなことある･･･？って気もしています。不明です。

とりあえずfloat32になってたり64になってたりしそうなところの第一候補は_gen_hello_hiho_queryで、↓みたいな感じでfloat32にすれば両方とも値が合うかもです。
https://github.com/tarepan/voicevox_engine/blob/c67bc2f26363f58d1de65279fb95b1cb3ac92ef9/test/tts_pipeline/test_tts_engine.py#L420-L421

あるいはもう雑に他と同様にround_floats(round_value=2)にしちゃう手もあるかもです。
upspeak辺りで += 0.3 する処理がありますが、0.3はfloat32とfloat64で微妙に値が違って計算誤差が出ると思います。
float32とfloat64が交じる場合は、round_value=5は難しいのかもしれません。

[github-actions]

Coverage Result

Resultを開く

Name	Stmts	Miss	Cover
run.py	508	318
voicevox_engine/init.py	1	0
voicevox_engine/cancellable_engine.py	94	72
voicevox_engine/core/core_adapter.py	81	29
voicevox_engine/core/core_initializer.py	59	30
voicevox_engine/core/core_wrapper.py	257	183
voicevox_engine/dev/core/init.py	2	0
voicevox_engine/dev/core/mock.py	36	4
voicevox_engine/dev/tts_engine/init.py	2	0
voicevox_engine/dev/tts_engine/mock.py	28	0
voicevox_engine/engine_manifest/EngineManifest.py	36	0
voicevox_engine/engine_manifest/EngineManifestLoader.py	12	0
voicevox_engine/engine_manifest/init.py	3	0
voicevox_engine/library_manager.py	92	4
voicevox_engine/metas/Metas.py	36	0
voicevox_engine/metas/MetasStore.py	28	6
voicevox_engine/metas/init.py	2	0
voicevox_engine/model.py	180	9
voicevox_engine/morphing.py	71	46
voicevox_engine/part_of_speech_data.py	5	0
voicevox_engine/preset/Preset.py	13	0
voicevox_engine/preset/PresetError.py	2	0
voicevox_engine/preset/PresetManager.py	80	2
voicevox_engine/preset/init.py	4	0
voicevox_engine/setting/Setting.py	11	0
voicevox_engine/setting/SettingLoader.py	17	0
voicevox_engine/setting/init.py	3	0
voicevox_engine/tts_pipeline/acoustic_feature_extractor.py	34	0
voicevox_engine/tts_pipeline/kana_converter.py	88	1
voicevox_engine/tts_pipeline/mora_list.py	7	0
voicevox_engine/tts_pipeline/text_analyzer.py	146	6
voicevox_engine/tts_pipeline/tts_engine.py	264	84
voicevox_engine/user_dict.py	145	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	13	0
voicevox_engine/utility/path_utility.py	26	8
voicevox_engine/utility/run_utility.py	10	7
TOTAL	2446	822

[tarepan]

float32だったりfloat64だったり ... 一応実行環境のアーキテクチャーに依存

貴重な情報ありがとうございます。

あるいはもう雑に他と同様にround_floats(round_value=2)にしちゃう

👍
影響要素がありすぎるので、round_value=2 でラフに解決します。

テストパスを確認しました。Reviewよろしくお願いします。

[Hiroshiba]

LGTM！！！