-
Notifications
You must be signed in to change notification settings - Fork 126
[Rust] SynthesisEngine::replace_mora_data を実装 #178
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
[Rust] SynthesisEngine::replace_mora_data を実装 #178
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!
@qwerty2501 さんはまだレビュー中という認識で合ってますか👀
PHONEME_MAP.len() | ||
} | ||
|
||
pub fn space_phoneme() -> String { |
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.
将来的に音素はEnumにすると型の恩恵受けられそうだなと思いました!
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
* implements SynthesisEngine::replace_mora_data * implements acoustic_feature_extractor * Internal, SynthesisEngine, InferenceCore からなる構造に変更 * implements replace_phoneme_length and replace_mora_pitch * resolve clippy warnings * allow dead_code * trivial change
内容
SynthesisEngine::replace_mora_data
を実装します。また、このメソッドの実装にはSynthesisEngine::replace_phoneme_length
およびSynthesisEngine::replace_mora_pitch
も必要なので、これらのメソッドも実装します。SynthesisEngine::replace_mora_data
SynthesisEngine::replace_phoneme_length
SynthesisEngine::replace_mora_pitch
関連 Issue
ref #128