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

テキスト欄内でルビを設定できるようにする #1383 #1384

Merged
merged 18 commits into from
Jul 12, 2023
Merged

テキスト欄内でルビを設定できるようにする #1383 #1384

merged 18 commits into from
Jul 12, 2023

Conversation

weweweok
Copy link
Contributor

内容

テキスト欄内でルビを設定できるようにする #1383

関連 Issue

ref #1383

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

  • テキスト書き出し
    image

テキストを繋げて書き出し
image

設定で「.txtを書き出し」をONにした状態で音声書き出し
image
image
image

  • 音声再生

空欄のテキスト欄に入力した後に再生
https://github.com/VOICEVOX/voicevox/assets/100256521/4d0ef4a7-eda7-4209-8e7a-3130cadfcf25

テキスト欄に何かを入力した状態で、変更した後に再生
https://github.com/VOICEVOX/voicevox/assets/100256521/453a2106-5418-447b-b646-50a92a6ded00

その他

音声再生の時に空白を入れているところがありますが、こうしないとなぜか読み上げなくなります。

@weweweok weweweok requested a review from a team as a code owner July 11, 2023 01:36
@weweweok weweweok requested review from Hiroshiba and removed request for a team July 11, 2023 01:36
@weweweok
Copy link
Contributor Author

メモ

export function skipWritingPart(targettext: string) {
  //{..|..}の前半部分をスキップ
  let resolvedText = targettext.replace(/\{(.*?)\|/g, "");
  resolvedText = resolvedText.replace(/\|(.*?)\}/g, "$1");
  return resolvedText;
}

export function skipReadingPart(targettext: string) {
  //{..|..}の後半部分をスキップ
  let resolvedText = targettext.replace(/\|.*?\}/g, "");
  resolvedText = resolvedText.replace(/\{(.*?)\|/g, "$1");
  resolvedText = resolvedText.replace(/\{/g, "");
  return resolvedText;
}

@Hiroshiba
Copy link
Member

Hiroshiba commented Jul 11, 2023

PRありがとうございます!!

今の実装だと、{あああ|と半分しか記法がない場合でも処理してしまいそうですね!(それはそれで便利かもしれませんが)
あとresolvedText.replace(/\{/g, "");によって{が全部消えていそうです。

ChatGPT君にちょっと実装を聞いてみました、参考になれば!!
https://chat.openai.com/share/d8b56439-787f-4702-9e26-e16c323c3164

@weweweok
Copy link
Contributor Author

@Hiroshiba

ありがとうございます!GPTに対する指示の的確さが凄すぎます...!!

  • テキスト書き出し
    image

テキストを繋げて書き出し
image
image

設定で「.txtを書き出し」をONにした状態で音声書き出し
image
image
image

  • 音声再生

空欄のテキスト欄に入力した後に再生
https://github.com/VOICEVOX/voicevox/assets/100256521/53715685-948d-4d52-821f-6772891197e5

テキスト欄に何かを入力した状態で、変更した後に再生
https://github.com/VOICEVOX/voicevox/assets/100256521/5614bf22-1247-466e-b204-8070560a8a1a

{あああ|と半分しか記法がない場合でも処理してしまいそうですね!

出力いただいたコードだとこのようになりますがどうでしょうか?
https://github.com/VOICEVOX/voicevox/assets/100256521/d7bedf11-c19e-45c8-9a59-58d4fd53ed62

src/store/audio.ts Outdated Show resolved Hide resolved
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!!!

いろいろチェックとか動画で撮影とかありがとうございました!!!
またプルリクエストいただけるととても心強いです、ぜひ・・・!!!

@Hiroshiba
Copy link
Member

細かいところですが書き方が揃っていなかったので揃えてみました。
マージします!!

@Hiroshiba Hiroshiba merged commit f3ac324 into VOICEVOX:main Jul 12, 2023
4 checks passed
@weweweok
Copy link
Contributor Author

weweweok commented Jul 12, 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.

2 participants