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] エンジンプロトタイプでレンダリングできるようにする #1595

Merged

Conversation

sigprogramming
Copy link
Contributor

@sigprogramming sigprogramming commented Oct 3, 2023

内容

エンジンプロトタイプでレンダリングできるようにします。
また、無声化を解除する処理の追加や、母音・子音の長さを編集する処理の修正などを行います。

OpenAPI Generatorの実行と、singing.tstype.tsの編集を行っています。

関連 Issue

VOICEVOX/voicevox_project#15
VOICEVOX/voicevox_engine#712

その他

@sigprogramming sigprogramming requested a review from a team as a code owner October 3, 2023 15:14
Comment on lines +988 to 1002
// 母音の後ろに子音がある場合
mora.vowelLength = noteLength - nextMora.consonantLength;
if (mora.vowelLength < minVowelLength) {
// 母音をこれ以上短くできない場合は、子音を短くする
mora.vowelLength = minVowelLength;
nextMora.consonantLength = noteLength - mora.vowelLength;
if (nextMora.consonantLength < minConsonantLength) {
// 子音も短くできない場合は、比で母音と子音の長さを決定する
const minMoraLength = minVowelLength + minConsonantLength;
const vowelRatio = minVowelLength / minMoraLength;
const consonantRatio = minConsonantLength / minMoraLength;
mora.vowelLength = noteLength * vowelRatio;
nextMora.consonantLength = noteLength * consonantRatio;
}
}
Copy link
Contributor Author

@sigprogramming sigprogramming Oct 3, 2023

Choose a reason for hiding this comment

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

母音+子音の長さがノートの長さを超えないようにします。

Copy link
Member

Choose a reason for hiding this comment

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

コメントが適量書いてあってものすごい読みやすかったです!!
かなり単純な調整だと思うのですがちょっと触ってみてかなり十分に感じました。すごい。

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

すごい!!!!!!!!!!!!

@Hiroshiba Hiroshiba merged commit aa06c65 into VOICEVOX:project-s Oct 7, 2023
6 checks passed
@sigprogramming sigprogramming deleted the support_for_engine_prototype branch October 14, 2023 16:25
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