C++ TTSライブラリをmainにマージ
#104
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* add C++ engine * add on/off clang-format * flatten phoneme * remove main * fix windows error * remove new in openjtalk * fix type warning * return vector and use calloc and free * use pragma once * fix miss * format * use namespace * Update core/src/engine.cpp Co-authored-by: qwerty2501 <[email protected]> * Update core/src/engine.cpp Co-authored-by: qwerty2501 <[email protected]> * Update core/src/engine.cpp Co-authored-by: takana-v <[email protected]> * add submodule option Co-authored-by: qwerty2501 <[email protected]> Co-authored-by: takana-v <[email protected]>
* erase cout * delete include iostream
* C API定義の改善と実装を行った(既存実装に影響でない範囲で) - voicevox_tts内でエラー発生してもユーザーが知ることができないのでResultCode用のenumを定義し、戻り値にした - voicevox_ttsでもともとの戻り値だった音声データのポインタはそのさらにポインタを関数の引数として受け取って成功時に出力するように変更した - ResultCodeをメッセージに変換する関数を定義 - C言語では extern "C" を使えないのでC++を使ってるときのみ extern"C"するように変更 - VOICEVOX_CORE_APIは関数宣言時のみにあれば良いので使い終わったらundefするように変更 - engine.cpp内でthrowしてるところがあったが、他言語ではC++の例外をcatchすることができないので例外が発生したらアプリケーションが確定で終了してしまう。そのため例外を投げるのではなくError Result Codeを返すように変更した * binary_sizeについては書き換わるとのことなのでポインタに戻した * 軽微な修正 - enumの定数を大文字に - outputを意図する引数にprefix outputを追加
newしてそのまま開放されていないものがあったのでnew使うのをやめた
* add dict loaded variable and getter * default contain openjtalk in synthesis engine * Update core/src/engine/synthesis_engine.h Co-authored-by: qwerty2501 <[email protected]> * fix miss (remove asterisk) * remove load dictionary constructor * remove load dict constructor * update func name * fix return code enum and error message * initialize synthesis engine in global * format Co-authored-by: qwerty2501 <[email protected]>
OpenJTalkで管理している各フィールドはコピーできてしまうと二重でclarが実行されたり、あるいはメモリリークが発生する可能性がある これを避けるために[Copy Protection(あるいはCopy Guard)というテクニック](https://ykoreeda.blogspot.com/2014/08/c.html) を使うことによりOpenJTalkがコピーされるようなコードが書かれるとコンパイルエラーになるように実装した
* aquestalk notation * format * add extract one character func * use extract one character func and fix bug * fix segmentation fault * fix pause delimiter/no pause delimiter bug * refactor (remove double quote) * refactor * use size_t * format * fix func name
# Conflicts: # CMakeLists.txt # core/CMakeLists.txt
|
LGTM！！
|
|
なんかrebase and mergeはエラーになったので普通のmergeで･･･！ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
C++ TTSライブラリ作業用ブランチを
mainにマージします。これにより、
mainブランチにTTS機能群(通常TTS/AquesTalkライク記法TTS)が実装されます。関連 Issue
close #43
その他
C++ TTSライブラリブランチは複数人でコミットを行ったので、squshではなく通常のマージをしたいです。