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.

Sign up for GitHub

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

Jump to bottom

feat!: "話者" ("speaker") → "キャラクター" ("character") #943

Merged
merged 2 commits into from
Jan 25, 2025
Merged

feat!: "話者" ("speaker") → "キャラクター" ("character") #943

merged 2 commits into from
Jan 25, 2025

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Jan 25, 2025

内容

ソングを見据え、エンジンの内部実装のように「キャラクター」としてしまう。

これでspeaker_uuidと(何故か残っている)speaker_id ("話者ID")以外、"speaker"/"話者"という表現は消える。

BREAKING-CHANGE: SpeakerMeta → CharacterMeta。
BREAKING-CHANGE: SpeakerVersion → CharacterVersion。

関連 Issue

Refs: #931 (review)

その他

@qryxip qryxip requested a review from Hiroshiba January 25, 2025 09:06
Hiroshiba
Hiroshiba approved these changes Jan 25, 2025
View reviewed changes
Copy link
Member

@Hiroshiba Hiroshiba left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM！！

･･･別PRとかで、speaker_idもcharacter_idだかidだかに変更しちゃっても良いかもですね！
エンジン側はエンジン側で変換しちゃえば良いのと、まあ0.16は結構大きな変更なのでついでに。

利点は「わかりやすくなる」、欠点は「エンジンとずれる」「今までのデータ構造とずれる」かな。

（VVMはすぐ作り直せるのと、あとエンジンが0.16コアに対応するまでは0.15時代のコア用binファイルと並列でメンテしていくつもりなのでそっちは大丈夫･･･なはず！ 🙏 ）

@qryxip
Copy link
Member Author

qryxip commented Jan 25, 2025

･･･別PRとかで、speaker_id

speaker_uuidの方ですかね？

「エンジンとずれる」

ちょっとこれが大きいかなと思いました。以前にENGINEを触ったユーザーが混乱しそう。「これは古くからある名前で、今は"キャラクター"と呼んでいる存在のUUID」ということを案内できればいいんじゃないかと。

@qryxip qryxip merged commit 8112ecb into VOICEVOX:main Jan 25, 2025
31 checks passed
@qryxip qryxip deleted the feat-rename-speaker-to-character branch January 25, 2025 15:21
@Hiroshiba
Copy link
Member

Hiroshiba commented Jan 26, 2025 •
edited
Loading

なるほどです！

まーーーよほどのことがない限り、シリアライズ可能なデータ構造を考えるのも大変ですしエンジンに合わせておく判断もいいかもですね！！

@qryxip qryxip mentioned this pull request Jan 26, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@qryxip @Hiroshiba