Add: トラック毎の書き出しを追加 #2228

sevenc-nanashi · 2024-08-18T00:43:42Z

内容

stem書き出し機能を追加します。

書き出し
ファイル名設定

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

その他

（なし）

Hiroshiba

draft PRありです！！
お待ちしています･･･！

Hiroshiba · 2024-08-18T07:27:34Z

src/store/singing.ts

@@ -2252,6 +2258,148 @@ export const singingStore = createPartialStore<SingingStoreTypes>({
    ),
  },

+  // TODO: EXPORT_WAVE_FILEとコードが重複しているので、共通化する


共通化やっぱり難しそうですかね･･･？

ん～どうでしょう、でもあんまり共通化できそうなところもないので消しても良さそう...？

たぶん音声出力は３箇所目なのとかなり長いのとで、流石に共通化できると嬉しそう。
いずれ書き出し設定したあとに書き出すダイアログを作るときに共通化しないといけなくなるはずだし。

まあでもぎりぎりこのプルリクのタイミングじゃなくても良さそう。
してあげといた方が後続の助けになりそうだな〜って感じかなと！
（このままだとマルチトラック書き出しがどんどん複雑になって、新しい実装をする前に交通整理が必要になる。マルチエンジンみたいに。）

sevenc-nanashi

一通りできました。

src/components/Dialog/SettingDialog/SettingDialog.vue

src/components/Dialog/HotkeyRecordingDialog.vue

Hiroshiba

提案２つあるので２つコメントしました！
（スレッド分けたかったので適当なファイルにコメントしました）

Hiroshiba · 2024-08-18T14:17:53Z

src/components/Dialog/HotkeyRecordingDialog.vue

トラックごとの書き出し機能、人によっては何度も何度も実行すると思うので、ショートカットキー実装しちゃうのどうでしょう？

ソングとトークで同じショートカットキーを設定出来ないっぽいので、それの修正と一緒に別PRでやろうとおもいます。

うおーたすかります！！

Hiroshiba · 2024-08-18T14:19:06Z

src/components/Dialog/HotkeyRecordingDialog.vue

何度も修正＆書き出しする場合、たぶん「書き出し先を固定する」とかなり相性が良さそうなのですが、どうでしょう･･･？

ソング側でショートカットキーで書き出し
→　DAW側で勝手に再読込される（されるんだろうか）
→　聞く
→　ソング側で微調整
→　以下繰り返し

実際あったら嬉しいですね。
#1250
これと一緒に実装…？

組み合わせ良さそうですね！！
そのissueはUI/UXが自明じゃないので、少なくともこのPR内でのついでに実装はレビューが厳しくなる予感があります。
とりあえずそちらのissueについては、まずはissue内でUI/UXの検討を進めるとかどうでしょ？

ですね。

src/components/Dialog/SettingDialog/DialogCell.vue

Hiroshiba

細かいとこ含めてコメントしてみました！

export waveを共通化すべきかとか辺りまだ見れてないです🙇

src/components/Dialog/SettingDialog/FileNamePatternDialog.vue

src/components/Dialog/SettingDialog/SettingDialog.vue

src/components/Sing/menuBarData.ts

src/store/singing.ts

src/store/utility.ts

src/type/preload.ts

Hiroshiba · 2024-08-31T07:42:32Z

あっすみません！　こちら再レビューしても大丈夫そうでしょうか 👀

sevenc-nanashi · 2024-08-31T09:33:07Z

あ～Request Review忘れてました

Hiroshiba

ちょっと細かいとこが多いですが、プログラミング周りのことをコメントしました！！
あんま興味なければ言っていただければ🙇

src/components/Dialog/SettingDialog/DialogButtonCell.vue

src/components/Dialog/SettingDialog/FileNamePatternDialog.vue

src/components/Dialog/SettingDialog/SettingDialog.vue

Hiroshiba · 2024-09-01T12:01:05Z

src/store/singing.ts

@@ -2252,6 +2258,148 @@ export const singingStore = createPartialStore<SingingStoreTypes>({
    ),
  },

+  // TODO: EXPORT_WAVE_FILEとコードが重複しているので、共通化する


たぶん音声出力は３箇所目なのとかなり長いのとで、流石に共通化できると嬉しそう。
いずれ書き出し設定したあとに書き出すダイアログを作るときに共通化しないといけなくなるはずだし。

まあでもぎりぎりこのプルリクのタイミングじゃなくても良さそう。
してあげといた方が後続の助けになりそうだな〜って感じかなと！
（このままだとマルチトラック書き出しがどんどん複雑になって、新しい実装をする前に交通整理が必要になる。マルチエンジンみたいに。）

src/store/utility.ts

Hiroshiba

storybookのテスト落ちてるの何でなんだろ。。。。

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts

This reverts commit 153a0c3.

Hiroshiba

ほぼLGTMです！！
パッチもありがとうございます！！

package.json

patches/@storybook+test-runner+0.19.1.patch

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>

Hiroshiba · 2024-09-10T11:52:18Z

patches/@storybook+test-runner+0.19.1.patch

結局postinstallが必要で申し訳ない😇

ちょっとrootにディレクトリ増やしたくない思いがあり、patchesディレクトリを移せないかと思ってます。
buildディレクトリ以下に置いても良いでしょうか？
（ドキュメント読んだ感じ --patch-dir で変更できそう）

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts

Hiroshiba

ほぼLGTMです！！
もろもろありがとうございました！！

コメントしましたが、ご面倒であればこっちでやっとこうと思います🙏

sevenc-nanashi · 2024-09-10T12:21:51Z

反映しました｡

Hiroshiba

LGTM！！！

長らくありがとうございました！！！
良い機能だと思います！！

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts

Hiroshiba · 2024-09-10T15:26:41Z

こんな感じで明日のお昼頃にツイートさせていただこうと思います！

書き出し時ダイアログもぜひ･･･！！

あ、あとそういえば特に問題が報告されていないので、次回のアップデートでマルチトラックの実験的機能を外そうかなと考えています！
Discord の方でちょっと話題に上げてみます！

Add: トラック毎書き出しを追加

671f52c

sevenc-nanashi requested a review from a team as a code owner August 18, 2024 00:43

sevenc-nanashi requested review from Hiroshiba and removed request for a team August 18, 2024 00:43

sevenc-nanashi marked this pull request as draft August 18, 2024 00:43

Hiroshiba reviewed Aug 18, 2024

View reviewed changes

sevenc-nanashi added 2 commits August 18, 2024 20:08

Add: ファイル名設定を追加

d93e31b

Add: パラメータ適用の切り換えを追加

e4b4a3f

sevenc-nanashi marked this pull request as ready for review August 18, 2024 11:21

sevenc-nanashi commented Aug 18, 2024

View reviewed changes

src/components/Dialog/SettingDialog/SettingDialog.vue Outdated Show resolved Hide resolved

src/components/Dialog/HotkeyRecordingDialog.vue Outdated Show resolved Hide resolved

Hiroshiba reviewed Aug 18, 2024

View reviewed changes

Delete: パラメータ書き出し周りを削除

ffa5035

Hiroshiba reviewed Aug 26, 2024

View reviewed changes

src/components/Dialog/SettingDialog/DialogCell.vue Outdated Show resolved Hide resolved

Hiroshiba reviewed Aug 26, 2024

View reviewed changes

sevenc-nanashi added 7 commits August 26, 2024 21:51

Change: デフォルトの書き出しファイル名にキャラ名を追加

74a34dd

Code: applyTrackParameters -> shouldApplyTrackParameters

456fa43

Change: DialogCell -> DialogButtonCell

7a6d6c6

Change: throwする

ed17385

Code: スタイル修正

f41ac76

Code: buildFileName -> fileNameBuilder

9166ab7

Refactor: 共通化

ab3e0ec

sevenc-nanashi requested a review from Hiroshiba August 31, 2024 09:32

Hiroshiba reviewed Sep 1, 2024

View reviewed changes

sevenc-nanashi added 5 commits September 2, 2024 13:41

Change: DialogButtonCell -> EditButtonCell

02e619f

Change: propsを変える

fdff200

Code: 定義を移動

eac3d15

Change: click -> buttonClick

9d7e724

Change: ファイル名のフォーマットあたりを変える

e1c5ac6

sevenc-nanashi added 2 commits September 6, 2024 19:07

Code: コメントを追加

e9f2b7c

Update: test-runnerを更新してみる

2cb5f55

Hiroshiba reviewed Sep 6, 2024

View reviewed changes

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts Outdated Show resolved Hide resolved

Add: patchを追加

0c64861

Hiroshiba reviewed Sep 7, 2024

View reviewed changes

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts Outdated Show resolved Hide resolved

sevenc-nanashi added 7 commits September 8, 2024 06:20

Fix: fnを消す

d1599af

Delete: patchを消す

153a0c3

Revert: patchを戻す

a9c8729

This reverts commit 153a0c3.

Delete: fnを消す

145835a

Fix: patchしてない範囲があったので修正

c340ba8

Add: コメントを追加

ee1f05f

Delete: 不要な分岐を削除

456d1d8

Hiroshiba reviewed Sep 9, 2024

View reviewed changes

sevenc-nanashi and others added 7 commits September 9, 2024 18:10

Code: patchのコメントを足す

f91ab93

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>

Change: findByTextにしてみる

b1b3055

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>

Change: patch-packageを明示的に実行しないようにする

e2cde6d

Refactor: clearAndInputに共通化

532e2f2

Fix: awaitする

9dcc8f8

Change: toBeInTheDocumentにする

423e4c3

Fix: やっぱりpatch-packageする

b610fe4

Hiroshiba reviewed Sep 10, 2024

View reviewed changes

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts Show resolved Hide resolved

Hiroshiba reviewed Sep 10, 2024

View reviewed changes

sevenc-nanashi added 2 commits September 10, 2024 21:18

Change: patchesを移動

a8956a2

Code: コメントを追加

6754aac

Hiroshiba approved these changes Sep 10, 2024

View reviewed changes

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts Outdated Show resolved Hide resolved

src/components/Dialog/SettingDialog/FileNameTemplateDialog.stories.ts Outdated Show resolved Hide resolved

Apply suggestions from code review

dd7fae6

Hiroshiba merged commit 051c6f9 into VOICEVOX:main Sep 10, 2024
9 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add: トラック毎の書き出しを追加 #2228

Add: トラック毎の書き出しを追加 #2228

sevenc-nanashi commented Aug 18, 2024 •

edited

Loading

Hiroshiba left a comment

Hiroshiba Aug 18, 2024

sevenc-nanashi Aug 18, 2024

Hiroshiba Sep 1, 2024

sevenc-nanashi left a comment

Hiroshiba left a comment •

edited

Loading

Hiroshiba Aug 18, 2024

sevenc-nanashi Aug 26, 2024

Hiroshiba Aug 26, 2024

Hiroshiba Aug 18, 2024 •

edited

Loading

sevenc-nanashi Aug 18, 2024

Hiroshiba Aug 18, 2024

sevenc-nanashi Aug 18, 2024

Hiroshiba left a comment

Hiroshiba commented Aug 31, 2024

sevenc-nanashi commented Aug 31, 2024

Hiroshiba left a comment

Hiroshiba Sep 1, 2024

Hiroshiba left a comment

Hiroshiba left a comment

Hiroshiba Sep 10, 2024

Hiroshiba left a comment

sevenc-nanashi commented Sep 10, 2024

Hiroshiba left a comment

Hiroshiba commented Sep 10, 2024

Add: トラック毎の書き出しを追加 #2228

Add: トラック毎の書き出しを追加 #2228

Conversation

sevenc-nanashi commented Aug 18, 2024 • edited Loading

内容

関連 Issue

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

その他

Hiroshiba left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sevenc-nanashi left a comment

Choose a reason for hiding this comment

Hiroshiba left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Hiroshiba Aug 18, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Hiroshiba left a comment

Choose a reason for hiding this comment

Hiroshiba commented Aug 31, 2024

sevenc-nanashi commented Aug 31, 2024

Hiroshiba left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Hiroshiba left a comment

Choose a reason for hiding this comment

Hiroshiba left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Hiroshiba left a comment

Choose a reason for hiding this comment

sevenc-nanashi commented Sep 10, 2024

Hiroshiba left a comment

Choose a reason for hiding this comment

Hiroshiba commented Sep 10, 2024

sevenc-nanashi commented Aug 18, 2024 •

edited

Loading

Hiroshiba left a comment •

edited

Loading

Hiroshiba Aug 18, 2024 •

edited

Loading