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

整理: talk/sing キャラクターの取得をメソッドへ移動 #1418

Merged
merged 1 commit into from
Jun 22, 2024

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented Jun 22, 2024

内容

概要: talk/sing キャラクターの取得をメソッドへ移動してリファクタリング

現在の ENGINE では talk/sing キャラクターの取得(全取得 + フィルタリング)は router で実装されている。
sing 機能が後から追加された関係でこれらは router に実装されているが、本来は MetasStore に実装されるべきである。こうすれば speaker/singer といった literal 値を内部実装に閉じ込めておけるし、filter_characters_and_styles() の利用範囲をより限定できる。また #1391 での CoreManager 内部化をおこなうのであればメソッド化は必須である。

このような背景から、talk/sing キャラクターの取得をメソッドへ移動するリファクタリングを提案します。

本 PR はコード移動に特化した PR です。

関連 Issue

ref #1391

@tarepan tarepan requested a review from a team as a code owner June 22, 2024 17:06
@tarepan tarepan requested review from Hiroshiba and removed request for a team June 22, 2024 17:06
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!!

そういえば新規参入者はmetasがなにのmetasか分からなそうなので、CharacterMetasとかにすると混乱抑えられるかも・・・?

@Hiroshiba Hiroshiba merged commit 94acb66 into VOICEVOX:master Jun 22, 2024
4 checks passed
@tarepan tarepan deleted the refactor/metas_talk_sing_method branch June 23, 2024 02:18
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