VOICEVOX / voicevox_core

無料で使える中品質なテキスト読み上げソフトウェア、VOICEVOXのコア
https://voicevox.hiroshiba.jp/
MIT License
857 stars 115 forks source link

`Synthesizer`の構造改革をする #685

Open qryxip opened 9 months ago

qryxip commented 9 months ago

内容

Synthesizserの改革を行います。

https://github.com/VOICEVOX/voicevox_core/pull/553#issuecomment-1656366892

  1. SynthesisEngineを解体し、Synthesizser内に
  2. SynthesizserSynthesizser<impl Borrow<OpenJTalk> + ...> | Synthesizser<()>
  3. APIの浅い所 (compatible_engineに至るまで)にndarrayを要求

関連 Issue

545

その他

Hiroshiba commented 9 months ago

PRありがとうございます!

1600行の更新、なるほどです。 おそらくコードの移動が大半になっているので、そこだけでもプルリクエストを分離していただけると助かるかもしれません。 移動だけなら差分をレビューすることが簡単なのですが、移動に変更が加わっていると難度が跳ね上がってしまうので・・・

ただまあ、今回は仕方ないということにして、APIの設計があまり変わっていないのであれば、ほぼレビューなしでマージすることも可能かもしれません。 Googleのエンジニアプラクティスに、変更は小さくしてプルリクを投げた方が良いという思想があります。 コミッター・レビュワー双方にメリットがあるというロジックなのでよかった読んでみてください! https://google-engineering-practices.translation.shuuji3.xyz/review/developer/small-cls.html