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

mainブランチとのconflict解消がうまく行ってなかったのでリトライ #206

Merged
merged 18 commits into from
Jul 24, 2022

Conversation

qwerty2501
Copy link
Contributor

内容

T/O

関連 Issue

refs #128

その他

絶対にSquash mergeしないでください!

qwerty2501 and others added 18 commits May 9, 2022 02:58
本当はデフォルト引数を消したかったが、使ってる人がいる可能性があるためC++で使ってる場合はそのままにするように修正した
* load_model関数を切り出す

* load_modelとis_model_loadedを足した

* 使われてないエラーメッセージを消去

* Update core/src/core.cpp

Co-authored-by: qwerty2501 <[email protected]>

* Update core/src/core.cpp

Co-authored-by: qwerty2501 <[email protected]>

* Update core/src/core.cpp

Co-authored-by: qwerty2501 <[email protected]>

* pythonのラッパーの型を変更

* load_all_models追加

* return true

Co-authored-by: qwerty2501 <[email protected]>
* Catch2をプロジェクト導入して参考となる1ケースを追加した

* リリース用ビルドにはテストビルドは不要のため無効化する変数を追加した

* テストコードの短縮化を行った

* READMEにテスト実行方法を追加

* テストをビルドするかどうかのフラグをデフォルトOFFにした

* テストをビルドするかどうかのフラグを変更したことによりREADMEを修正

* MSVCに対して足りないオプションを追加

- utf8文字列でコンパイルするように指定
- C++20を明示的に指定
- __cplusplusの値を利用中のC++のバージョンに合わせるように指定(ないとC++98相当になるとか)

* Configがリリースの場合のみに最適化オプションを指定するように修正

* オプションの指定を短くまとめた

* coreの標準ライブラリもバージョンアップした

* compile optionsの修正

* Catch2にもCXX_STANDARD 20を追加

* Windows環境でビルドできるように設定を修正

VOICEVOX#121 (comment)

* format 効いてしまっていたところを修正
* .gitignore open_jtalk_dic in example

* modify: example/python/.gitignore
* C++サンプルコードへのリンクを追加

* #readme

* Update README.md
* コード署名

* build_util

* artifact/

* a

* remove
* inputのbooleanは文字列として渡ってくるので判定を修正

* 修正もれ
…y2501/voicevox_core into fix/conflict-main-for-rust
@qwerty2501 qwerty2501 mentioned this pull request Jul 24, 2022
Copy link
Member

@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

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

絶対にSquash mergeしないでください!

すみませんでした! 今後気をつけます。

@PickledChair
Copy link
Member

Squash and merge 以外の選択肢がないので、 @Hiroshiba さんに頼むことになりそうでしょうか……?

squash_and_merge_only

@qwerty2501
Copy link
Contributor Author

わすれないように目立たせてただけだったんですが、今見返すと強い言葉になってますねw

@Hiroshiba
Copy link
Member

普通のマージします!

@Hiroshiba Hiroshiba merged commit 1284e26 into VOICEVOX:rust Jul 24, 2022
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.

6 participants