Skip to content

SynthesisEngineInferenceCoreを解体する#686

Merged
Hiroshiba merged 1 commit into
VOICEVOX:mainfrom
qryxip:decompose-synthesis-engine-and-inference-core
Nov 21, 2023
Merged

SynthesisEngineInferenceCoreを解体する#686
Hiroshiba merged 1 commit into
VOICEVOX:mainfrom
qryxip:decompose-synthesis-engine-and-inference-core

Conversation

@qryxip

@qryxip qryxip commented Nov 19, 2023

Copy link
Copy Markdown
Member

内容

SynthesisEngineInferenceCoreを解体し、そのコードをSynthesizserに押し込めます。

#553 (comment)

SynthesizserStatusOpenJtalkを直接持つという形になります。

また元のコードは原則そのまま移しているのですが、関数を関数内関数にする、という程度のことは行っています。

関連 Issue

#545

その他

@qryxip qryxip mentioned this pull request Nov 19, 2023
69 tasks
Comment thread crates/voicevox_core/src/engine/synthesis_engine.rs

@Hiroshiba Hiroshiba left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!!!

移動無視のgit diffコマンドを使って差を見比べていました、大丈夫そう!!

いたるところに、FIXMEコメントだけ書き足すだけにし、リファクタリングは一切しないという配慮を感じました!
差分とても見やすかったです、ありがとうございます!!

一箇所だけコメントしていますが、問題なさそうだったらマージしちゃっていただけると!

@Hiroshiba Hiroshiba merged commit 1befb27 into VOICEVOX:main Nov 21, 2023
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.

3 participants