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

experimental_katakana_transcription オプションの追加 #834

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kdrkdrkdr
Copy link

@kdrkdrkdr kdrkdrkdr commented Dec 8, 2023

内容

概要: experimental_katakana_transcription オプションの追加
デフォルト値はFalseに設定されています。

追加コメント

ko2kanaを別々にパッケージに分離しておきました。

Co-authored-by: takana-v <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>

## 内容
概要: experimental_katakana_transcription オプションの追加
デフォルト値はFalseに設定されています。

## 追加コメント
[ko2kana](https://github.com/kdrkdrkdr/ko2kana)を別々にパッケージに分離しておきました。
@kdrkdrkdr kdrkdrkdr requested a review from a team as a code owner December 8, 2023 16:10
@kdrkdrkdr kdrkdrkdr requested review from Hiroshiba and removed request for a team December 8, 2023 16:10
@kdrkdrkdr kdrkdrkdr changed the title Master experimental_katakana_transcription オプションの追加 Dec 8, 2023
@Hiroshiba
Copy link
Member

PRありがとうございます!! 個人的には面白そうだなと感じています。

ちなみにファイル容量はどれくらい増えそうでしょうか?
pip install ko2kanaする前とした後のsite_packagesディレクトリのサイズを比較すればわかるのかなと思っています!

@kdrkdrkdr
Copy link
Author

PRありがとうございます!! 個人的には面白そうだなと感じています。

ちなみにファイル容量はどれくらい増えそうでしょうか? pip install ko2kanaする前とした後のsite_packagesディレクトリのサイズを比較すればわかるのかなと思っています!

戦後比較したとき、約300KBくらいになります。

@takana-v
Copy link
Member

テストがエラーで落ちてしまっているみたいです。
https://github.com/VOICEVOX/voicevox_engine/actions/runs/7143609752/job/19471588986?pr=834

以下のコマンドで修正できるはずです。

python -m pip install -r requirements-test.txt
pysen run format lint

@Hiroshiba
Copy link
Member

Hiroshiba commented Dec 10, 2023

@kdrkdrkdr ありがとうございます!!

比較したとき、約300KBくらいになります。

おお!それくらいであれば全然大丈夫そうです!!
あ、これは英語も含まれていますか?
英語の辞書ファイルを含めるともっと増えそうだなーと・・・!

@kdrkdrkdr
Copy link
Author

테스트가 오류로 떨어지는 것 같습니다. https://github.com/VOICEVOX/voicevox_engine/actions/runs/7143609752/job/19471588986?pr=834

다음 명령으로 수정할 수 있어야 합니다.

python -m pip install -r requirements-test.txt
pysen run format lint

requirements-test.txtにはko2kanaを入れておかなくて発生するようです。

@kdrkdrkdr
Copy link
Author

@kdrkdrkdr감사합니다! !

비교했을 때, 약 300KB 정도가 됩니다.

오! 그 정도라면 전혀 괜찮을 것 같습니다! ! 아, 이게 영어도 포함되나요? 영어의 사전 파일을 포함하면 좀더 늘어날 것 같다····!

英語はすでに含まれていますが、これは他のパッケージであるg2p_enが従属性としてインストールされる必要があります。

@Hiroshiba
Copy link
Member

これは他のパッケージであるg2p_enが従属性としてインストールされる必要があります。

なるほどです! ちょっと調べた感じこちらのライブラリはwheelが3MBくらいでした! まあOKかな・・・?

@kdrkdrkdr
Copy link
Author

こちらは大丈夫ですが、もしかして問題になったりしますか?

@Hiroshiba
Copy link
Member

こちらは大丈夫ですが、もしかして問題になったりしますか?

あ、こちらも大丈夫だと思います!

@kdrkdrkdr
Copy link
Author

コードには問題がないのに今flake8ですごくかかりますね(×﹏×)
修正してまた持って来なければならないようですね。

@kdrkdrkdr
Copy link
Author

flake8 disableする方法はありませんか?
注釈とあれこれコードが長いと思います..
機能はちゃんと作動しますが···
これは本当に分からないですね..

@Hiroshiba
Copy link
Member

どうしてもflake8をdisableしたい部分は、その行の最後にnoqaとコメントすればOKです!
run.py内に何箇所かnoqaがあるので参考になると思います!

@kdrkdrkdr
Copy link
Author

問題が解決されたようですが、レビューしていただけますか?

@takana-v
Copy link
Member

ko2kanaの1.8が存在していないため、エラーが発生しているようです。

ERROR: Could not find a version that satisfies the requirement ko2kana==1.8 (from versions: 1.1, 1.2, 1.3, 1.4, 1.5, 1.6, 1.7)
ERROR: No matching distribution found for ko2kana==1.8
Error: Process completed with exit code 1.

@kdrkdrkdr
Copy link
Author

ごめんなさい···うっかりして1.8バージョンをpypiにアップロードしませんでした··· 今アップロードしましたので、もう一度確認していただけますか?

@tarepan tarepan mentioned this pull request Jan 3, 2024
Copy link
Contributor

@tarepan tarepan left a comment

Choose a reason for hiding this comment

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

👍 VOICEVOX の新たな展開を可能にする興味深い機能です、実装もコンパクトで実験機能として適切ですね。

@Hiroshiba
CIテスト履歴が消えているので、メンテナ権限で CI re-run は可能でしょうか?

@Hiroshiba
Copy link
Member

@tarepan mainブランチをマージさせていただいたらテストが回りました!
@kdrkdrkdr さんも、もし作業を再開される場合は最初にpullしていただければ! 🙇

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.

5 participants