-
Notifications
You must be signed in to change notification settings - Fork 200
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
リファクタリング: SynthesisEngineをモジュール化 #214
リファクタリング: SynthesisEngineをモジュール化 #214
Conversation
Pull Request Test Coverage Report for Build 1535336244
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
手元で動かしてみましたが特に問題ないように思いました。
将来的にはSynthesisEngine部分をcoreに移動もしくは別リポジトリ化するかもしれないので、その場合にも役に立つかなと思いました。
ref VOICEVOX/voicevox_core#43
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
相対参照は煩雑になりがちで好きじゃなかったのですが、モジュール化を目指すなら確かに早退パスにした方が良いなと感じました。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
内容
SynthesisEngineをモジュール化(
__init__.py
でimportするように)します。また、
make_synthesis_engine
関数を切り出します。SynthesisEngine内で使われているmora関連の関数の切り出しは別PRに分割しています。
関連 Issue
ref #199
ref #119