Skip to content
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

feat: いくつかのAPIを露出し、「テキスト音声合成の流れ」を明確に #1025

Merged

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Feb 27, 2025

内容

次のAPIをパブリックAPIとして露出させ、それに関連するテストとドキュメントを用意する。このテストとドキュメントが主目的。テストでは各ショートハンドがショートハンドとして機能するか確かめ、ドキュメントにおいてはテキスト音声合成の流れというものを用意してMermaidのflowchart図を載せる。また各ショートハンドメソッドについて、何のショートハンドなのかを明記するようにする。

  • OpenJtalk::analyze/TextAnalyzer::analyze
    (Rust APIでは既に露出済み)
  • Synthesizer::open_jtalk/Synthesizer::text_analyzer
  • AudioQuery::from_accent_phrases

kana系のAPIについては今回はノータッチ。

関連 Issue

その他

@qryxip qryxip marked this pull request as draft February 27, 2025 18:04
@qryxip qryxip force-pushed the pr/feat-expose-some-apis-and-explain-tts-process branch 3 times, most recently from 5da56b3 to 56ee34b Compare February 27, 2025 18:50
@qryxip qryxip marked this pull request as ready for review February 27, 2025 18:50
@qryxip qryxip requested a review from Hiroshiba February 27, 2025 18:50
@qryxip qryxip force-pushed the pr/feat-expose-some-apis-and-explain-tts-process branch from 56ee34b to 3157748 Compare February 27, 2025 19:18
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!!!

ドキュメント見て、確かに結構案内してあげると良さそうなことはあるなぁと感じました。
0.16にはなくていいかもだけど、例えばそもそも音素長はなんなのかとか、アクセントは何なのかとか。
・・・・・本格的に解説しようとすると大変すぎる!!

@qryxip
Copy link
Member Author

qryxip commented Mar 1, 2025

  • 8e948df (#1025): ショートハンドメソッドに対するテストにおいて、Synthesizer::create_accent_phrasesを含めてなかったので追加。ちなみにこれによりPython APIにあるバグを発見。後でPR出します。
  • f7a4ebf (#1025): C APIテストのtts_via_audio_queryttsにリネーム

@qryxip
Copy link
Member Author

qryxip commented Mar 1, 2025

#1026

@qryxip
Copy link
Member Author

qryxip commented Mar 1, 2025

↑の更新については軽微なので、一度approve貰ったということでマージしようと思います。

@qryxip qryxip merged commit 741f6e3 into VOICEVOX:main Mar 1, 2025
36 checks passed
@qryxip qryxip deleted the pr/feat-expose-some-apis-and-explain-tts-process branch March 1, 2025 07:54
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.

2 participants