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

C++サンプルコードへのリンクを追加 #142

Merged
merged 4 commits into from
Jun 9, 2022

Conversation

Hiroshiba
Copy link
Member

内容

C++サンプルコードへのリンクがREADMEになくてもったいなかったので、追記しました。

その他

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.

ちょっと細かい指摘な気がするのですが、提案として1つコメントしました。

もしそのままで良いという判断であればそのままマージしても良いと思います……!

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@PickledChair
Copy link
Member

すみません、言いそびれていたのですが、Approve 済みです……!

@Hiroshiba
Copy link
Member Author

マージします!

@Hiroshiba Hiroshiba merged commit c47d055 into main Jun 9, 2022
@Hiroshiba Hiroshiba deleted the add-cpp-exapmle-link-to-readme branch June 9, 2022 09:28
PickledChair pushed a commit that referenced this pull request Jul 24, 2022
* デフォルト引数はC言語では使えないのでC++のみに有効になるように変更した (#122)

本当はデフォルト引数を消したかったが、使ってる人がいる可能性があるためC++で使ってる場合はそのままにするように修正した

* initializeで全モデルを読み込まなくても良いようにした (#124)

* 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]>

* workflow_dispatchでもビルド開始可能に (#127)

* ビルド時にREADMEファイルを追加する (#131)

* append unit testing framework (#121)

* Catch2をプロジェクト導入して参考となる1ケースを追加した

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

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

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

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

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

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

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

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

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

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

* compile optionsの修正

* Catch2にもCXX_STANDARD 20を追加

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

#121 (comment)

* format 効いてしまっていたところを修正

* static castやnullopt比較をやめ、value関数などを使うようにする (#118)

* use value and has value

* remove has_value

* coreのビルド時にバージョン情報がちゃんと
入るようにする

* hotfix イントネーションの推論をCPUで行うように (#146)

* python example for 0.12, update FFI (#138)

* .gitignore open_jtalk_dic in example (#148)

* .gitignore open_jtalk_dic in example

* modify: example/python/.gitignore

* C++サンプルコードへのリンクを追加 (#142)

* C++サンプルコードへのリンクを追加

* #readme

* Update README.md

* コード署名できるようにする (#164)

* コード署名

* build_util

* artifact/

* a

* remove

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

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

* 修正もれ

Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Yuto Ashida <[email protected]>
Co-authored-by: Yosshi999 <[email protected]>
Co-authored-by: nebocco <[email protected]>
y-chan pushed a commit to SHAREVOX/sharevox_core that referenced this pull request Sep 1, 2022
* C++サンプルコードへのリンクを追加

* #readme

* Update README.md
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