-
Notifications
You must be signed in to change notification settings - Fork 205
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
pyopenjtalkのバージョンを更新 #469
pyopenjtalkのバージョンを更新 #469
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
実際に日本語パスでも動くか確認したいのですが、良い方法無いですかね・・・。 |
以下の voicevox_engine/voicevox_engine/utility/path_utility.py Lines 22 to 26 in 341a46f
|
たしかに・・・!ちょっと試してみようかなと思います! |
試してみた感じいけてそうでした! ↓diff # FIXME: ファイル保存場所をエンジン固有のIDが入ったものにする
# FIXME: Windowsは`voicevox-engine/voicevox-engine`ディレクトリに保存されているので
# `VOICEVOX/voicevox-engine`に変更する
- return Path(user_data_dir("voicevox-engine"))
+ return Path(user_data_dir("voicevox-engine-ヒホ")) ↓バージョン $ pip show pyopenjtalk | grep Version
Version: 0.2.0+f4ade2 このエンジンを起動した状態でエディタを起動し、辞書登録→その辞書が使われていることを確認した感じです。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
#469 (comment) の結果があり、多分大丈夫だと思うのでマージしてしまいます……! |
requirements更新
内容
pyopenjtalkをVOICEVOX/pyopenjtalk@f4ade29 に更新します。
関連 Issue
close #438
スクリーンショット・動画など
その他
コメントが増えているのはrequestモジュール追加時の更新漏れが原因だと思われます。