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

Add: ユーザー辞書を追加 #40

Merged
merged 3 commits into from
Nov 15, 2023

Conversation

sevenc-nanashi
Copy link
Member

内容

ユーザー辞書を実装します。
動くことは確認しています(追加ボタンをユーザー辞書ダイアログ表示ボタンにしてテストした)

関連 Issue

スクリーンショット・動画など

(なし)

その他

(なし)

@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner November 15, 2023 11:43
@sevenc-nanashi sevenc-nanashi requested review from y-chan and removed request for a team November 15, 2023 11:43
Comment on lines 98 to 100
// Rustのtempfileクレートのための設定。
// /data/local/tmp はAndroid 10から書き込めなくなった。そのため、
// filesのディレクトリ内に一時フォルダを用意してそこから書き込むように設定する。
Copy link
Member Author

@sevenc-nanashi sevenc-nanashi Nov 15, 2023

Choose a reason for hiding this comment

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

ここら辺はあとでCoreにドキュメントPR出します。

@@ -17,7 +17,7 @@
"adjust_intonation_scale": true,
"adjust_volume_scale": true,
"interrogative_upspeak": true,
"synthesis_morphing": true,
"synthesis_morphing": false,
Copy link
Member Author

Choose a reason for hiding this comment

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

これのせいで一部バグっていたので直しました。

@@ -37,6 +37,8 @@ export type VoicevoxCorePlugin = {
speakerId: number;
enableInterrogativeUpspeak: boolean;
}) => Promise<{ value: string }>;

useUserDict: (obj: { wordsJson: string }) => Promise<void>;
Copy link
Member Author

Choose a reason for hiding this comment

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

UUIDとかの情報は必要ないので単語だけを毎回渡す形にしています。

@Hiroshiba Hiroshiba requested review from Hiroshiba and removed request for y-chan November 15, 2023 16:14
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!!!

ラッパー部分はしっかりレビューしてきていませんが、スピード優先ということでマージさせていただきます!!

Comment on lines -19 to +23
corePlugin.initialize().then(() => {
(async () => {
await corePlugin.initialize();
await useUserDictWords(corePlugin, await getUserDictWords());
isCoreInitialized = true;
});
})();
Copy link
Member

Choose a reason for hiding this comment

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

起動シーケンスに組み込んでちゃんとエンジンが起動してることを保証してあげるようにしたいですね~

@Hiroshiba Hiroshiba merged commit 571fd07 into VOICEVOX:main Nov 15, 2023
7 checks passed
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