Skip to content

Conversation

misogihagi
Copy link
Contributor

内容

wheelを利用するとさらの状態からの実行がやりやすくなりました。
その手順がイマイチわかりにくかったのでコマンドを添えておきました。

関連 Issue

#414

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.

改良ありがとうございます、助かります!!

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.

@misogihagi 結構wheel版のPythonサンプルを通ろうとして、同じようにREADMEの不足で困られている方が多い印象です。

頂いているPRでREADMEをしっかり整備してみませんか・・・! 👀

あとは別PRで頂いたDownloader周りもちょっと追従して、wheelのURLの案内辺りを整理すれば完成かなと思っています!!

@misogihagi
Copy link
Contributor Author

misogihagi commented Mar 28, 2023

#446 の件はもともと抱えていたものでどうしようもない気はしますが…
とりあえず使用しているスクリプトはrustのものに変えたのと、
リリースのダウンローダーのところ、wheelみたいに環境ごとに分けた表を作って欲しいです!

@Hiroshiba
Copy link
Member

@misogihagi たしかにそうですね、書き足してみました!
ちょっと最新版はまだ整備できてないので、とりあえず0.14.1にしてます。
https://github.com/VOICEVOX/voicevox_core/releases/tag/0.14.1

@qryxip
Copy link
Member

qryxip commented Apr 24, 2023

@misogihagi すみません、コンフリクト解消はこちらでしてしまいました!
(別の方のこれと間違えてしまいました...)

@qryxip
Copy link
Member

qryxip commented Apr 24, 2023

@misogihagi お忙しいようでしたらこのPRも後はこちらでやらせて頂こうかなと思っています。いかがでしょうか?
(@Hiroshiba 大丈夫でしょうか?)

@Hiroshiba
Copy link
Member

PR開発者の方からOKをいただくか、数日待って返信がないなどであれば大丈夫です・・・!

@misogihagi
Copy link
Contributor Author

misogihagi commented Apr 26, 2023

@qryxip
いろいろ決めあぐねていたので代わりに書いてもらえるならありがたいです!

@qryxip qryxip mentioned this pull request Apr 27, 2023
@qryxip qryxip requested a review from Hiroshiba May 5, 2023 09:45
@qryxip
Copy link
Member

qryxip commented May 6, 2023

コンフリクト解消はしましたが、#475で"open-jtalk-dict-dir"のデフォルト値が"./voicevox_core/open_jtalk_dic_utf_8-1.11"になっているので少しおかしなことになっているかも。

@qryxip
Copy link
Member

qryxip commented May 6, 2023

いや--dict-dir自体は特におかしくはないですね。CWDに辞書ではなく

  1. run.pyと
  2. onnxruntime.dll

の両方が同時に無いといけなくて、今のreadmeだとそうなっていない。

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 and others added 2 commits May 11, 2023 00:01
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
qryxip and others added 4 commits May 11, 2023 00:03
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Ryo Yamashita <qryxip@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
qryxip and others added 2 commits May 17, 2023 00:12
Co-authored-by: Ryo Yamashita <qryxip@gmail.com>
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!!

実際にREADMEに従ってlinux環境で試せることも確認しました! 良いですね!!

@Hiroshiba
Copy link
Member

テスト待ちです、テスト通ったらマージしていただけると。

@qryxip 引き継ぎありがとうございました!!

@PickledChair
Copy link
Member

テスト通ったのでマージしてしまいます。 @misogihagi さんと、途中から引き継いだ @qryxip さん感謝です……!
(私の macOS 環境でも手順通りで動作しました)

@PickledChair PickledChair merged commit 67b3938 into VOICEVOX:main May 18, 2023
@qryxip qryxip mentioned this pull request May 18, 2023
Hiroshiba pushed a commit that referenced this pull request May 19, 2023
@qryxip qryxip mentioned this pull request May 22, 2023
67 tasks
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.

4 participants