We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
現在進行している・予定している Speaker/Metas 系リファクタリングの方針・段階・進捗をここに記す。
内部での Speaker インスタンス扱いがトリッキーで性能低下の原因にもなっている。 API BaseModel である Speaker に縛られていることがトリッキーさの原因であるため、内部都合に合わせた内部クラスを用意する。 また Speaker 関連コードが MetasStore 以外の関数やベタ書きアドホック処理に分散しているため、MetasStore へ集約し丁寧にテストする。それに合わせ CoreManager を内蔵する(TTSEngine を参照)。 語彙確認が完了したため、上記リファクタリングは新しい語彙を用いておこなう。
Speaker
MetasStore
CoreManager
TTSEngine
CoreAdapter.speakers
_speaker_info()
.load_combined_metas()
Character
.talk_styles
.sing_styles
filter_speakers_and_styles()
CoreSpeaker
load_combined_metas()
SpeakerInfo
list[Speaker]
generate_app()
The text was updated successfully, but these errors were encountered:
dataclass
tarepan
No branches or pull requests
内容
現在進行している・予定している Speaker/Metas 系リファクタリングの方針・段階・進捗をここに記す。
方針
内部での
Speaker
インスタンス扱いがトリッキーで性能低下の原因にもなっている。API BaseModel である
Speaker
に縛られていることがトリッキーさの原因であるため、内部都合に合わせた内部クラスを用意する。また
Speaker
関連コードがMetasStore
以外の関数やベタ書きアドホック処理に分散しているため、MetasStore
へ集約し丁寧にテストする。それに合わせCoreManager
を内蔵する(TTSEngine
を参照)。語彙確認が完了したため、上記リファクタリングは新しい語彙を用いておこなう。
段階と進捗
CoreAdapter.speakers
出力をモデル化 #1260_speaker_info()
をMetasStore
へ移植 #1261.load_combined_metas()
引数を変更 #1352整理:MetasStore
にCoreManager
を内蔵 #1314Character
内部型を追加 #1320Speaker
ライクで.talk_styles
/.sing_styles
を持ったCharacter
内部クラスを新設filter_speakers_and_styles()
を非破壊動作へCharacter
ベースでリライト_speaker_info()
を簡略化 #1322_speaker_info()
内のフィルタリングを find 系でリライトCoreSpeaker
の強制Speaker
化を削除 #1365Character
利用範囲を拡大Character
利用範囲を拡大 #1385load_combined_metas()
戻り値、filter_speakers_and_styles()
引数SpeakerInfo
系ファイル読み込みを async で並列化検討list[Speaker]
の内部キャッシュを検討MetasStore
をgenerate_app()
引数化して DI 可能にThe text was updated successfully, but these errors were encountered: