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

[project-s] シーケンスの追加・削除をチャンネル内で行わないようにする #1536

Merged
merged 4 commits into from
Sep 11, 2023

Conversation

sigprogramming
Copy link
Contributor

@sigprogramming sigprogramming commented Sep 4, 2023

内容

チャンネル周りのリファクタリングです。
シーケンスの追加・削除をチャンネル内で行わないようにします。
(チャンネルに音を調整する機能のみ持たせるようにします)
また、名前をSingChannelからChannelStripに変更して、AudioRenderer.tsに移動します。
チャンネルに持たせていたSequenceとAudioPlayer・SynthはひとまずPhraseに持たせる形にします。

関連 Issue

ref #1041

その他

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!!

いつか、どのクラスがなにを担当しているかの概念図がほしくなりそうだな~と感じました。
まあ設計が定まってきたら・・・!!

@sigprogramming
Copy link
Contributor Author

レビューありがとうございます!
設計が定まってきたらドキュメントを書いていこうと思います…!

Copy link
Contributor

@romot-co romot-co left a comment

Choose a reason for hiding this comment

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

@sigprogramming
遅れまして申し訳ございません…!今回詳細見られていないのですが、LGTMです!

@Hiroshiba Hiroshiba merged commit 18a86c7 into VOICEVOX:project-s Sep 11, 2023
@sigprogramming sigprogramming deleted the refactor_channel branch October 14, 2023 16:24
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.

3 participants