Skip to content

Conversation

PickledChair
Copy link
Member

@PickledChair PickledChair commented Jul 14, 2022

内容

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

@PickledChair PickledChair marked this pull request as ready for review July 14, 2022 20:50
Copy link
Member

@Hiroshiba Hiroshiba left a 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 {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

将来的に音素はEnumにすると型の恩恵受けられそうだなと思いました!

Copy link
Contributor

@qwerty2501 qwerty2501 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Hiroshiba Hiroshiba merged commit 19b7455 into VOICEVOX:rust Jul 15, 2022
qwerty2501 pushed a commit to qwerty2501/voicevox_core that referenced this pull request Jul 23, 2022
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants