-
Notifications
You must be signed in to change notification settings - Fork 122
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
example下の"speaker_id"を、"style_id"に直す #562 #584
Conversation
READMEの記述というと?
これなら消さない方がいいと思いますね、 #582 などの破壊的変更の議論もありますし |
example/python/README.mdの ここと voicevox_core/example/python/README.md Line 55 in e571d70
ここです voicevox_core/example/python/README.md Line 66 in e571d70
|
あ、それは更新するべきですね( |
uintptr_t output_binary_size = 0; | ||
uint8_t* output_wav = nullptr; | ||
VoicevoxTtsOptions ttsOptions = voicevox_make_default_tts_options(); | ||
|
||
result = voicevox_synthesizer_tts(synthesizer,wide_to_utf8_cppapi(speak_words).c_str(), speaker_id, ttsOptions, &output_binary_size, &output_wav); | ||
result = voicevox_synthesizer_tts(synthesizer,wide_to_utf8_cppapi(speak_words).c_str(),p style_id, ttsOptions, &output_binary_size, &output_wav); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p style_id
ここ修正ミスかも?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
1箇所だけ多分訂正間違いがあると思うのでコメントしてみました。
uintptr_t output_binary_size = 0; | ||
uint8_t* output_wav = nullptr; | ||
VoicevoxTtsOptions ttsOptions = voicevox_make_default_tts_options(); | ||
|
||
result = voicevox_synthesizer_tts(synthesizer,wide_to_utf8_cppapi(speak_words).c_str(), speaker_id, ttsOptions, &output_binary_size, &output_wav); | ||
result = voicevox_synthesizer_tts(synthesizer,wide_to_utf8_cppapi(speak_words).c_str(),style_id, ttsOptions, &output_binary_size, &output_wav); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
result = voicevox_synthesizer_tts(synthesizer,wide_to_utf8_cppapi(speak_words).c_str(),style_id, ttsOptions, &output_binary_size, &output_wav); | |
result = voicevox_synthesizer_tts(synthesizer,wide_to_utf8_cppapi(speak_words).c_str(), style_id, ttsOptions, &output_binary_size, &output_wav); |
空白を入れるべきだと思います
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
マージします(CI終わったら)
内容
#562
関連 Issue
ref #562
その他
README.mdの記載も変える必要がありますか?