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.

Sign up for GitHub

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

Jump to bottom

マルチトラック：mainにマージ #2185

Merged
merged 20 commits into from
Jul 30, 2024
Merged

マルチトラック：mainにマージ #2185

merged 20 commits into from
Jul 30, 2024

Conversation

sevenc-nanashi
Copy link
Member

@sevenc-nanashi sevenc-nanashi commented Jul 29, 2024 •
edited
Loading

内容

project-multitrackをmainにマージします。長く苦しい戦いだった（本当はまだ色々残ってるのでcloseじゃなくてrefにしてます）

関連 Issue

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

（なし）

その他

#2184 が先です。

sevenc-nanashi and others added 17 commits June 3, 2024 00:06
@sevenc-nanashi @Hiroshiba @sigprogramming
マルチトラック：storeを色々変える (#2093)
642a354 
* Change: phrase以外をtrackIdを受け取るようにする

* wip

* Update: singing.tsを色々変える

* Update: store以外を良い感じに合わせる

* Add: マイグレーションを追加

* Fix: SET_TRACKでselectedTrackIdが無を参照するのを修正

* Fix: 二重にセットするように

* Fix: trackOrderにする

* Change: SELECT_TRACKしないようにする

* Code: data -> pitchArray

* Code: SET_SINGING_GUIDE_KEY_TO_PHRASEに型をつける

* Code: こっちも型をつける

* WIP: むずい

* Change: PhraseにtrackIdを持たせる

* Fix: 型エラーを修正

* Change: SET_NOTESとかの処理をSET_TRACKに写す

* Fix: overlappingNoteIds.hasが抜けてたのを修正

* Change: initialTrackIdにする

* Code: コメントを修正

* Change: singerAndFrameRatesを最初に作っておく

* Code: トラック周りのactionを移動

* Fix: overlappingNoteInfos周りを修正

* Change: Mapのキーを変える

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Change: CREATE_TRACKをADD_TRACKと分割

* Code: 空行を開ける

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Change: overlappingNoteIdsをトラック毎に持つようにする

* Code: コメントを移動

Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com>

* Fix: Renderする

* Add: trackOrderに入ってるかバリデーションする

* Add: tracksに入ってるかバリデーションする

* Delete: SET_TRACK_ORDERを一度消す

* Delete: singerAndFrameRatesの片引き数を消す

* Add: trackのバリデーションを追加

* Fix: overlappingNoteIdsを代入してなかったので修正

* Change: 判定方法を変更

* Change: tracks.hasに一本化する

---------

Co-authored-by: Hiroshiba <Hiroshiba@users.noreply.github.com>
Co-authored-by: sigprogramming <sigprogramming@users.noreply.github.com>
@sevenc-nanashi
Merge: main -> buffer/project-multitrack/20240620-solve
0d07c12
@Hiroshiba
マルチトラック：mainをマージ (#2125)
12abe27 
## 内容

project-multitrackのbufferにmainをマージします。

## 関連 Issue

（なし）

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

（なし）

## その他

（なし）
@sevenc-nanashi @Hiroshiba
マルチトラック：コンポーネントを分離する (#2129)
f05d174 
* Refactor: コンポーネントを分離

* Fix: エンジンアイコン周りの変更に追従

* Fix: engineIconsの引数を修正

* Apply suggestions from code review

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
@sevenc-nanashi @Hiroshiba
マルチトラック：インポートに対応 (#2126)
7771ae9 
* Change: SET_TRACKを作る

* Add: インポートをマルチトラック対応

* Add: 複数トラックをインポートできるように

* Fix: indexじゃなくてiだった

* Add: 共通化 + undo出来るように

* Change: トラック選択のUIを変更

* Change: mapを使う

* Change: isTracksEmptyを移動

* Change: IMPORT_*_PROJECTをCOMMANDに

* Add: isValidTrackチェックを追加

* Change: QCheckboxを使う

* Change: 見た目を良い感じにする

* デザインちょっと調整

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
@sevenc-nanashi
Merge: main -> buffer/project-multitrack/20240701
12ce49c
@Hiroshiba
マルチトラック：mainをマージ (#2150)
2ca0723 
## 内容

mainをマルチトラックのブランチにマージします。

## 関連 Issue

（なし）

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

（なし）

## その他

（なし）
@Hiroshiba
マルチトラック：mainをマージ (#2159)
a2d4da8 
## 内容

project-multitrackにmainを持ってきます。主に #2128 です。

## 関連 Issue

（なし）

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

（なし）

## その他

conflictが無かったのでmainから直接投げています。一応buffer作った方が良いかも？
@sevenc-nanashi @github-actions @Hiroshiba
マルチトラック：サイドバーを追加 (#2148)
63dbe09 
* Add: とりあえず移植

* Add: 型とかを追加

* Add: トラック追加できるように

* Add: actionを追加

* Improve: 見た目を良い感じにする

* Add: utaformatixの変換にnameを含める

* Add: テストを追加

* Add: TODOを追加

* Delete: overlappingNoteInfosあたりを削除

* Add: フォールバックを追加

* Add: コメントを沢山追加

* Add: コメントをちょっと追加

* Add: コメントをもうちょっと追加

* Change: singer-name -> track-name

* Delete: z-indexに頼らないようにする

* Improve: CSS周りを整理

* Add: character周りをまとめる

* add: cloneWithUnwrapProxyに包む

* Add: 設定出来るように

* Update: スナップショット更新 [update-snapshots]

* Update: スナップショット更新 [update snapshots]

* （スナップショットを更新）

* Change: コメントの位置を変える

* Change: QOptionGroupにする

* Change: プロパティ名を変える

* Refactor: 細かいところを良い感じにする

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Code: コメントを足す

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Code: コメントを足す

* Delete: isDraggingを消す

* Fix: 細かいところを修正

* （スナップショット更新）

[update snapshots]

* （スナップショットを更新）

* Delete: TODOを消す

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

* Code: 順番を揃える

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

* Delete: $qを消す

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

* Change: volume -> gain

* Change: singer-style -> singer-name

* Code: コメントを追加

* Change: heightの指定をSingEditorに移す

* Change: 影響を受けないようにするやり方を変える

* Change: サイドバーの幅を保存する

* Change: 幅を細かくする

* Delete: 余白を消す

* Change: content: strictに

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

* Change: watchEffectにする

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

* Delete: setTrackNameをなくす

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

* Change: コロンを使わないようにする

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

* Change: mutationの外でcloneWithUnwrapProxyを使う

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

* Add: watchEffectをインポートする

* Change: 警告を出すのは開発時だけにする

* Add: TODOを追加

* Change: 幅保存のロジックを変える

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Update src/components/Sing/SequencerPitch.vue

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Hiroshiba <Hiroshiba@users.noreply.github.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
@Hiroshiba
マルチトラック：mainをマージ (#2166)
5ab9bbe 
## 内容

いつものアレです。

## 関連 Issue

- ref: #2163 

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

（なし）

## その他

（なし）
@sevenc-nanashi @Hiroshiba
マルチトラック：Undo/Redoで壊れるのを修正 (#2165)
6d6ca16 
* Fix: stateの値に対してバリデーションをかける

* Change: NOTEにする

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

* Code: コメントを良い感じにする

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

* Change: hasにする

* Change: getSelectedTrackWithFallbackにする

* Change: NOTE_IDS -> ALL_NOTE_IDS

* Add: テストを追加

* Revert: 43fbc23.

* Change: current -> selected

* Change: SELECT_ALL_NOTES -> SELECT_ALL_NOTES_IN_SELECTED_TRACK

* Change: トラック削除時は1つ上に飛ぶようにする

* Code: コメントを良い感じにする

* Fix: dispatchの名前を修正

* Apply suggestions from code review

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
@sevenc-nanashi
マルチトラック：再生・書き出しに対応 (#2163)
f383998 
* WIP

* Add: とりあえず再生できるように

* Add: 全書き出しをマルチトラックに対応

* Change: previewSynthをglobalChannelStripに直接繋ぐようにする

* Change: generateDefaultSongFileNameをstoreの外に出す

* Change: watchStoreStatePluginにする

* Change: fileHelperから持ってくるようにする

* Change: convertToWavFileDataを外に出す

* Change: watchStoreStatePlugins -> watchStoreStatePlugin

* Code: コメントを追加

* Change: RENDER内を良い感じにする

* Delete: nextTickを削除

* Code: コメントの内容を変える

* Fix: 2回設定していたのを削除

* Change: globalChannelStrip -> mainChannelStrip

* Change: createAudioPlayerSequenceを戻す

* Change: shouldPlayTracks内のフィルタに任せる

* Change: getterにする

* Add: limiterを追加
@sevenc-nanashi @sigprogramming @Hiroshiba
マルチトラック：シーケンサー周りを対応 (#2161)
d9bbc78 
* Add: 他トラックのノートを出す様に

* Add: PhraseIndicatorを追加

* Fix: z-indexを修正

* Add: SequencerPitchを対応

* Delete: z-indexに頼らないようにする

* Code: コメントを動かす

* Change: line -> shadow-line

* Change: trackIdをnullableじゃなくする

* Change: notes -> allNotes

* Change: singingGuideKeysをフィルタする

* Change: singingGuideをフィルタするようにする

* Change: NoteにtrackIdを渡すのをやめる

* Change: trackIdのキー毎削除

* Change: hasOverlappingの表示切り替えをコンポーネント側でやる

Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com>

* Delete: singingGuidesを削除

Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com>

* Delete: 不要なnon null assertionを削除

Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com>

* Change: selectedNotes -> selectedNotesInCurrentTrack

Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com>

* Code: コメントを修正

* Code: 細かいところを修正

* Change: hasOverlappingError -> isOverlapping

Co-Authored-By: hiroshiba <hiroshiba@users.noreply.github.com>

* Change: 順番を揃える

Co-Authored-By: hiroshiba <hiroshiba@users.noreply.github.com>

* Change: current -> selected

Co-Authored-By: hiroshiba <hiroshiba@users.noreply.github.com>

* Change: selectedNoteIdsInCurrentTrackを消す

* Change: opacityを使わないようにする

* Delete: 不要な変数を削除

* Delete: TODOを消す

---------

Co-authored-by: sigprogramming <sigprogramming@users.noreply.github.com>
Co-authored-by: hiroshiba <hiroshiba@users.noreply.github.com>
@sevenc-nanashi
Merge: main -> buffer/project-multitrack/20240724
f71ff06
@Hiroshiba
マルチトラック：mainからマージ (#2177)
e57296e 
## 内容

main -> project-multitrackです。

## 関連 Issue

（なし）

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

（なし）

## その他

（なし）
@sevenc-nanashi @github-actions
@Hiroshiba @sigprogramming
マルチトラック：最終調整 (#2176)
2c87e5e 
* Change: shouldPlayTracksがSetを返すようにする

* Change: 実験的機能に隔離

* Change: 条件を修正

* Code: コメントを追加

* Add: TODOを追加

ついでに[update snapshots]

* （スナップショットを更新）

* Change: トグルの変数を変える

* Change: 抜け穴をふさぐ

* Delete: セーフガードをなくす

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

* Change: solo/muteの代入方法を変える

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

* Change: CLEAR_UNDO_HISTORYにする

* Change: disable周りを揃える

* Change: number | nullにする

* Change: findIndicesを消す

* Change: テキストをほんのり変える

* Fix: overlappingNoteIdsをコピーする

* Change: RENDERでChannelStripを更新するようにする

* Code: RENDERの前に空行を開ける

* Fix: 合成状態の順序を修正

* Fix: ピッチが見えなかったのを修正

* Change: toSortedをかける

* Change: SET_TRACKまわりを変える

* Fix: ソートするものが間違ってた

* Delete: watch周りのやつを削除

* Change: 変数名を変える

* Delete: 使われてない型を削除

* Code: 注意を追加

* Code: コメントを変える

* Code: コメントをいい感じにする

Co-authored-by: Sig <sigprogrammer@gmail.com>

* Change: SELECT_ALL_NOTES_IN_SELECTED_TRACK -> SELECT_ALL_NOTES_IN_TRACK

* Fix: 挙動を揃える

* Fix: ガバ

* Fix: Render忘れ

* Apply suggestions from code review

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Sig <sigprogrammer@gmail.com>
@sevenc-nanashi
Merge: main -> buffer/project-multitrack/20240730
548124c
@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner July 29, 2024 20:47
@sevenc-nanashi sevenc-nanashi requested review from Hiroshiba and removed request for a team July 29, 2024 20:47
Hiroshiba and others added 3 commits July 30, 2024 12:17
@Hiroshiba
マルチトラック：mainからマージ (#2184)
4834230 
## 内容

多分最後のconflict解消です。

## 関連 Issue

（なし）

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

（なし）

## その他

（なし）
@Hiroshiba
マルチトラック：detailsタグのコンフリクト解消ミス修正 (#2187)
ef16660
@sevenc-nanashi
マルチトラック：複数選択が出来ないのを修正 (#2186)
d5f6825 
Fix: 複数選択が出来ないのを修正
Hiroshiba
Hiroshiba approved these changes Jul 30, 2024
View reviewed changes
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/sing/domain.ts
Comment on lines -413 to +435
export function selectPriorPhrase(
phrases: Map<PhraseSourceHash, Phrase>,
export function selectPriorPhrase<K extends string>(
phrases: Map<K, Phrase>,
position: number,
): [PhraseSourceHash, Phrase] {
): [K, Phrase] {
Copy link
Member

Choose a reason for hiding this comment

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

この辺PhraseSourceHashだけに戻った気がするので、K extends stringじゃなくても良いかもですね
まあいいか。

@Hiroshiba Hiroshiba merged commit 7fbfde0 into main Jul 30, 2024
18 checks passed
@Hiroshiba Hiroshiba deleted the project-multitrack branch August 12, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sevenc-nanashi @Hiroshiba