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

保存するファイル名を 001_四国めたん_こんにちは.wav のようなものに変更する #7

Merged
merged 4 commits into from
Aug 4, 2021

Conversation

oov
Copy link
Contributor

@oov oov commented Aug 2, 2021

ファイル名が 001.wav だと後でファイルの内容がわかりづらくなってしまうので、
001_四国めたん_こんにちは.wav のような 通し番号_キャラ名_セリフ.wav の形式になるように変更しました。
セリフが長い場合は 001_四国めたん_長いセリフを喋って….wav のように9文字+ "…" の形式になります。

また、GENERATE_AND_SAVE_AUDIOfilePath を省略したときに表示される「名前を付けて保存」のダイアログでも、
同じ形式のファイル名が予め入力されている状態になるよう変更しました。

Hiroshiba
Hiroshiba previously approved these changes Aug 4, 2021
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!
制御文字の部分だけ追加して頂けると!

src/store/audio.ts Outdated Show resolved Hide resolved
Co-authored-by: Hiroshiba <[email protected]>
@oov
Copy link
Contributor Author

oov commented Aug 4, 2021

すみません、初めて使った機能で修正お送りしたら、そのまま誤って dismissed にしてしまいました。
レビューありがとうございます!

@Hiroshiba
Copy link
Member

すみません、こちらの設定ミスでdismissになってしまいました・・・ マージします!

@Hiroshiba Hiroshiba merged commit b8d6cfc into VOICEVOX:main Aug 4, 2021
@oov oov deleted the improve-save-filename branch August 5, 2021 18:57
takana-v pushed a commit that referenced this pull request Jul 20, 2022
* #1 PlayElectronアプリのインスタンス取得、タイトルのアサートを追加

* eslintの自動修正

* compare vue cli mode.

* #4 モンキーパッチなのでbackground.tsのエンジン連携部分をテストできていないがとりあえずデータ準備を超えられた。

* #7 package.jsonから削除。READMEのテスト実施方法も変更(暫定)。

* #7 cypressコンフィグとplaywrightコンフィグに入れ替え

* #7 削除忘れ

* #7 コードフォーマットの類

* #7 .github actionsの設定からCypressのキャッシュフォルダ指定を削除

* #7 package.jsonから削除。READMEのテスト実施方法も変更(暫定)。

* #7 cypressコンフィグとplaywrightコンフィグに入れ替え

* #7 削除忘れ

* #7 コードフォーマットの類

* VOICEVOX#803 test:e2eでE2Eテストを動かせるようにする

* so-c#6 electron-builder の非互換見落とし
sevenc-nanashi added a commit to sevenc-nanashi/voicevox that referenced this pull request Apr 30, 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