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

change: liberate VOICEVOX CORE #8

Merged
merged 3 commits into from
Sep 1, 2024

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Aug 30, 2024

内容

  1. SessionBuilder::commit_from_vv_binを追加し、暗号化対応ONNX Runtimeに暗号化ONNXを渡せるようにします。
  2. INFO以下のログをせき止めていたのをやめます。

関連 Issue

ref VOICEVOX/voicevox_project#24

スクリーンショット・動画など

その他

src/session/builder.rs Outdated Show resolved Hide resolved
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!!!

一箇所だけコメントしました!

@qryxip
Copy link
Member Author

qryxip commented Aug 31, 2024

VOICEVOX/voicevox_core#722 のため、libvoicevox_onnxruntimeではないようならば手前側でエラーとなるようにしました。
5b5b22a (#8)

@qryxip qryxip requested a review from Hiroshiba August 31, 2024 20:05
@qryxip qryxip force-pushed the change-liberate-voicevox-core branch from cb4acff to 5b5b22a Compare August 31, 2024 20:22
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.

マージします!
これがVOICEVOX由来のものかどうかのチェックの追加も良いと思いました!

vv_binで名称統一していく感じいいですね!

CStr::from_ptr(build_info)
.to_str()
.expect("should be UTF-8")
.starts_with("VOICEVOX ORT Build Info: ")
Copy link
Member

Choose a reason for hiding this comment

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

ここの文字列、大元の方を変えた時にこっちも変えるの忘れがちになるかもですね!
可能なら定数を定義して流用するのもありかも。
まあそんなに気にしなくてもいいと思うので、とりあえずこのプルリクエストではこのままで!

@Hiroshiba Hiroshiba merged commit 3ecf05d into VOICEVOX:main Sep 1, 2024
7 checks passed
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