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] 各Schedulerを直接使用してスケジューリングを行う形に変更する #1532

Merged

Conversation

sigprogramming
Copy link
Contributor

内容

音声周りのリファクタリングです。
AudioEventSchedulerとNoteEventSchedulerを直接使用してスケジューリングを行う形に変更します。
Schedulerの抽象化をやめて処理を追いやすくするのが目的です。

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

正直なところこの辺りは(以前から)半分ほどわかってないのですが、とりあえずスピード感優先で良いのかなと感じました!!

src/infrastructures/AudioRenderer.ts Outdated Show resolved Hide resolved
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.

ありがとうございます、LGTMです!
NoteとSoundの再生において抽象化した共通のSchedulerを介して扱っていたのを、
それぞれ直接利用するようにした形かと思っておりますが、誤解ありましたら失礼致します…!

@sigprogramming
Copy link
Contributor Author

レビューありがとうございます!
@romot-co おっしゃる形で間違いありません!

ループ実装時にまたこの辺り変更するかもですが、ひとまずリファクタということで、マージしたいと思います。

@sigprogramming sigprogramming merged commit ca98530 into VOICEVOX:project-s Sep 4, 2023
@sigprogramming sigprogramming deleted the refactor_scheduler 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