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

voicevox_synthesizer_audio_query を voicevox_synthesizer_create_audio_query にする #700

Closed
Hiroshiba opened this issue Dec 2, 2023 · 3 comments · Fixed by #882
Closed
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

内容

Synthesizerのメソッドの一つで、accent_phrasesを作るのはcreate_accent_phrasesなのに、audio_queryを作るのはcreate_audio_queryではなくaudio_queryとなっているので、そこの調整用issueです。
RustとPythonがaudio_queryになっていて、CとJavaがcreate_audio_queryになっているとのこと。

関連タスクの一つです。

Pros 良くなる点

一貫性が増す

Cons 悪くなる点

実現方法

変更してテストが回ればOK!
example内でも使われてる可能性があるのでチェックしておくとさらに良さそう。

その他

@Hiroshiba Hiroshiba added 機能向上 初心者歓迎タスク 初心者にも優しい簡単めなタスク labels Dec 2, 2023
@sevenc-nanashi
Copy link
Member

そういえば: #576 これで達成されてたのでCloseします。

@qryxip
Copy link
Member

qryxip commented Jan 19, 2024

RustとPythonがそのままですね…

@qryxip qryxip reopened this Jan 19, 2024
@sevenc-nanashi
Copy link
Member

おっと、ごめんなさい

qryxip added a commit that referenced this issue Dec 2, 2024
Resolves #700.

現状のC APIとJava APIと合わせる形で名前を統一する。理由としては:

1. 推論している感を出せる
    * 保証していない性質、例えば冪等性などを期待されにくくできる
2. `create_accent_phrases`と`create_audio_query`の二つをセットで目立たせ
   ることができる
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants