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

fix: ソロ全解除をして再生すると、全解除前のように聞こえるバグの修正 #2199

Conversation

Hiroshiba
Copy link
Member

内容

の修正プルリクエストです。

ついでに全singing.tsのactionを検査して、RENDERが必要そうなやつを点検してみました。
TIME_SIGNATURE周りでRENDERが呼ばれてないやつはいくつかあったんですか、これ多分いりますよね・・・?
ということでついでに入れときました。
COMMAND_の方にだけRENDERがなかった)

関連 Issue

close #2194

その他

点検してた感じ、多分これ使わないだろうなっていうactionがいっぱいありました。
コマンド以外ありえないコマンドじゃない関数とか。
呼び間違いがいつか起こりそうだ、消してもいいかもしれない。

@Hiroshiba Hiroshiba requested a review from a team as a code owner August 3, 2024 13:32
Copy link
Member

@sevenc-nanashi sevenc-nanashi 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

拍子はレンダリングに影響しないので、TIME_SIGNATURE周りはいらないかもです…!

@Hiroshiba
Copy link
Member Author

あっなるほどです!!!要らなそう!!

逆にCOMMANDじゃないTIME_SIGNATURE変更箇所には確かRENDERがあったので、そっち消しておきます!

@Hiroshiba
Copy link
Member Author

多分大丈夫だと思うのでマージします!

@Hiroshiba Hiroshiba merged commit 5e6693a into VOICEVOX:main Aug 4, 2024
9 checks passed
@Hiroshiba Hiroshiba deleted the ソロ全解除をして再生すると、全解除前のように聞こえるバグの修正 branch August 9, 2024 07:16
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