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

feat: pause_length{,_scale}をデフォルト値限定で受け入れる #872

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Nov 16, 2024

内容

AudioQuerypause_length{,_scale}を追加する。ただしそれぞれnull1.のみを許し、無音時間調整自体はまだ実装しない。

関連 Issue

Refs: VOICEVOX/voicevox_engine#1308, VOICEVOX/voicevox_engine#1425

その他

@qryxip qryxip requested a review from Hiroshiba November 16, 2024 02:24
@qryxip qryxip marked this pull request as draft November 16, 2024 10:16
@qryxip qryxip force-pushed the feat-accept-default-pause-length-and-pause-length-scale branch from 4271275 to 63153a1 Compare November 19, 2024 14:53
@qryxip qryxip marked this pull request as ready for review November 19, 2024 14:53
@qryxip qryxip marked this pull request as draft November 20, 2024 13:22
@qryxip
Copy link
Member Author

qryxip commented Nov 20, 2024

あ! 無音調整 やったの @rokujyushi さんじゃなくて @X-20A さんですね。大変失礼いたしました。
63153a1 (#872)

@qryxip qryxip force-pushed the feat-accept-default-pause-length-and-pause-length-scale branch 2 times, most recently from d2daaa5 to df00650 Compare November 20, 2024 13:31
VOICEVOX/voicevox_engine#1308VOICEVOX/voicevox_engine#1425 の一部
を参考にコードを書いた。

@Hiroshiba さんと以下の2名の許諾のもと、 VOICEVOX#874 にのっとりMITライセンスと
してライセンスする。

* @X-20A (VOICEVOX/voicevox_engine#1308)
* @sabonerune (VOICEVOX/voicevox_engine#1425)

Co-Authored-By: X-20A <[email protected]>
Co-Authored-By: sabonerune <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Refs: VOICEVOX#874 (comment)
Refs: VOICEVOX#874 (comment)
@qryxip qryxip force-pushed the feat-accept-default-pause-length-and-pause-length-scale branch from df00650 to c8deaad Compare November 21, 2024 06:26
@qryxip qryxip marked this pull request as ready for review November 21, 2024 10:05
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!!!

あ、1点だけ!
とりあえず型があるだけって感じですよね、ということは未実装なので一応未実装であることがわかるようなコメントがあった方がいいかも・・・?

ただもう多言語で実装されているのでコメントしていくの面倒かも。
であれば実装が完成してからこのプルリクエストマージする(実装がない期間をできる限り短くする)とかもありかも。

まあでもそんなに超重要というわけではないと思うので、お任せしたいと思います!

@qryxip
Copy link
Member Author

qryxip commented Nov 23, 2024

todo一つだけ入れてマージしようと思います。

@qryxip qryxip merged commit 38517cc into VOICEVOX:main Nov 23, 2024
31 checks passed
@qryxip
Copy link
Member Author

qryxip commented Nov 23, 2024

あ、Java忘れてた。まあ後で一緒にやれば…

@qryxip
Copy link
Member Author

qryxip commented Nov 24, 2024

↑ 別に忘れてなかった。やったのを忘れたっぽい…

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.

2 participants