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

エンジン側に実装された擬似的な疑問文に対応する #608

Closed
Hiroshiba opened this issue Dec 19, 2021 · 3 comments
Closed

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Dec 19, 2021

内容

エンジン側に、疑問文だった場合に語尾を上げる調整をすることで、擬似的に疑問文っぽい音声にする実装が加わりました

APIにenable_interrogativeという引数があり、こちらをtrue/falseにすることで擬似的な疑問文機能のon/offを切り替えられます。
APIはデフォルトでこちらがonになっていますが、エディタ的には不思議な挙動をしてしまう(語尾に伸ばし棒が加わる)ので、エディタ側はデフォルトでoffにしてあったほうが良いかなと感じます。
でも便利な機能なので、設定で切り替えられると嬉しそうです。

Pros 良くなる点

(擬似的に)疑問文が使える

Cons 悪くなる点

実現方法

流れはこんな感じかなと思います

VOICEVOXのバージョン

0.10.preview

@qwerty2501
Copy link
Contributor

これは設定のオプションに追加する感じですかね

@Hiroshiba
Copy link
Member Author

ですです!(おそらく)
実験的機能は現れたり消えたりするので、コメントアウトされていますが一応項目があります。

<!-- 今後実験的機能を追加する場合はここに追加 -->
<!-- FIXME: 0.9.1に間に合わなかったのでダークモード機能を一旦省きました -->
<!-- <q-card flat class="setting-card">
<q-card-actions>
<div class="text-h5">実験的機能</div>
</q-card-actions>
<q-card-actions class="q-px-md q-py-sm bg-setting-item">
<div>Theme</div>
<q-space />
<q-btn-toggle
unelevated
padding="xs md"
color="white"
text-color="black"
toggle-color="primary"
toggle-text-color="display"
v-model="currentThemeNameComputed"
:options="availableThemeNameComputed"
>
<q-tooltip
:delay="500"
anchor="center left"
self="center right"
transition-show="jump-left"
transition-hide="jump-right"
>
The colors in themes are not decided yet
</q-tooltip>
</q-btn-toggle>
</q-card-actions>
</q-card> -->

qwerty2501 added a commit to qwerty2501/voicevox that referenced this issue Dec 19, 2021
qwerty2501 added a commit to qwerty2501/voicevox that referenced this issue Dec 19, 2021
qwerty2501 added a commit to qwerty2501/voicevox that referenced this issue Dec 20, 2021
openapiのコード生成を行った

refs VOICEVOX#608 VOICEVOX#610
Hiroshiba pushed a commit that referenced this issue Dec 22, 2021
* feat: generated openapi code

openapiのコード生成を行った

refs #608 #610

* コード生成した結果、ビルドが通らなくなったので修正した

* openapitools.jsonをgit管理下に置くようにした
qwerty2501 added a commit to qwerty2501/voicevox that referenced this issue Dec 22, 2021
y-chan pushed a commit that referenced this issue Dec 28, 2021
* feat: 疑問文有効無効設定追加

refs #608

* feat: リクエスト時に疑問文自動調整有効設定値を送るようにした

refs #608

* feat: enableInterrogativeをSettingStoreState型の中でネストさせた

もしかしたらいらない配慮かもしれないが、
ExperimentalSettingオブジェクト型を作成し、その中にenableInterrogativeを持たせるようにした。
enableInterrogativeは現状実験的な機能だが、将来的に正式な機能として提供された場合デフォルト設定値をtrueにする可能性が高そうである。
これをSettingStoreStateに対してenableInterrogativeを直接持たせてそのままの状態でenableInterrogative機能が正式リリースされた場合、既存ユーザーはenableInterrogativeがfalseの状態のまま追加居続けるのではないか?
との疑念があったのでExperimentalSettingを作成してネストさせることにしてみた。もしかしたら違うかも。

こうすることによってenableInterrogative機能を正式リリースする際にstate.experimentalSetting.enableInterrogativeからstate.enableInterrogativeに構造を変えることによってデフォルト値trueを適用した状態で既存ユーザーにも機能を提供できるようになるはずである。

* feat: added enableInterrogative setting test case

* ExperimentalSettingとしてまとめて変更できるようにした!!!

* removed enableInterrogative args in AudioStoreTypes

stateを取れることがわかったため

* Update src/background.ts

必要っぽい?ので

Co-authored-by: Hiroshiba <[email protected]>

Co-authored-by: Hiroshiba <[email protected]>
@y-chan
Copy link
Member

y-chan commented Jan 4, 2022

solved by #610

@y-chan y-chan closed this as completed Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants