-
Notifications
You must be signed in to change notification settings - Fork 201
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
整理: Speaker
と内部型の分離
#1202
整理: Speaker
と内部型の分離
#1202
Conversation
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です!!
きれいな変更ありがとうございます!!!!
ちょっとデータの流れをより健全にできるからと思っていくつかコメントしてみました 🙇
@Hiroshiba |
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!!!
差分も綺麗だと感じました!
1点ちょっとどっちか迷ったのをコメントしました 🙇
@Hiroshiba |
内容
整理: VV API
Speaker
型と ENGINE 内部型(CoreSpeaker
・EngineSpeaker
)を分離してリファクタリングした。属性
SpeakerStyle
もCoreSpeakerStyle
として分離したため、VV API 話者情報と Core 話者情報は完全に分離された。これによりピッチ範囲情報を任意の箇所(ENGINE or CORE)に実装可能となる。
関連 Issue
ref #1121 (comment)