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

スタイルID(style_id)のことを話者ID(speaker_id)としているコードを全部置き換える #589

Closed
Hiroshiba opened this issue Jan 23, 2023 · 6 comments · Fixed by #826
Labels

Comments

@Hiroshiba
Copy link
Member

不具合の内容

昔APIで、スタイルIDを引数として受け取るところを、話者IDという引数名に設定しました。
この影響でコード全体としてスタイルIDなのに変数名がspeaker_idになっている箇所が散見され、いつかミスが起こりそうだなと感じました。

APIからもspeaker_idという名前をやめ、style_idに全体的にコードを変更したいです。

その他

互換性の破棄は大変なので、style_id/speaker_idともにOptionalにして受け取るのが良いかなと思っています。

@Hiroshiba Hiroshiba changed the title スタイルID(style_id)のことを話者ID(spekaer_id)としているコードを全部置き換える スタイルID(style_id)のことを話者ID(speaker_id)としているコードを全部置き換える Jan 23, 2023
@weweweok
Copy link
Contributor

weweweok commented Sep 5, 2023

取り組みます。

@weweweok
Copy link
Contributor

weweweok commented Sep 5, 2023

style_idとspeaker_idの明確な違いは、無いという認識で良いでしょうか?

@Hiroshiba
Copy link
Member Author

おお!よろしくお願いします!

speaker idとstyle idは明確に違いますが、一部誤用しているところがあるという感じです。
こちらに少し詳しく書いています。
https://github.com/VOICEVOX/voicevox/blob/main/docs/%E7%B4%B0%E3%81%8B%E3%81%84%E8%A8%AD%E8%A8%88%E6%96%B9%E9%87%9D.md#engineidspeakeridstyleid

@Hiroshiba
Copy link
Member Author

@weweweok さんありがとうございます!!!!
もうほぼ全域の表記間違いを修正できたと思います!!

チェックしているうちにいくつか見つけた追加の修正箇所があるのでここにメモしておきます。

  • README.md
    • speaker=style_idであるという紹介が不要になった
  • docs/VOICEVOX音声合成エンジンとの連携.md
    • speaker=style_idであるという紹介が不要になった
  • run.py
    • モーフィング周りが変更必要(base_speakerなど)

@Hiroshiba
Copy link
Member Author

メモです。
ドキュメント周りの修正は終わったので、後はrun.pyのモーフィング周りの変更だけかなと!

@MrHiraiwa
Copy link

いつも使わせてもらっています。外部のアプリからAPIつついて使っていたらリビルドしたタイミングで使えなくなって少し焦りました。それでここに辿り着きました。style_idからspeakerに戻して無事動作を確認しました。今後とも宜しくお願いします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants