Skip to content

Conversation

YCHuang2112sub
Copy link

asyncio.run() cannot be used in the jupyter notebook

asyncio.run() can not be called in jupyter-notebook, jupyter-lab
an notebook version demo of how to use the package, and also adding the sound playing function.
@tuna2134
Copy link
Member

🤖 このメッセージはAutoCodeReviewによって行われたコードレビューです 🤖

コードの解説:
・'gitignore'ファイルには、無視するディレクトリとファイルの設定が追加されています。
・'README.rst'ファイルでは、Jupyterノートブックでの実行時に'asyncio.run()'が使用できないため、代替のコードが追加されています。
・'basic.ipynb'ファイルは新しく追加され、VOICEVOXのシンプルなデモが含まれています。

コードの改善点:
・'gitignore'ファイルには、整理された設定が追加されています。
・'README.rst'ファイルでは、Jupyterノートブックで'asyncio.run()'が動作しない場合にも対応するための改善が行われています。
・'basic.ipynb'ファイルが追加され、VOICEVOXのデモがより簡単に実行できるようになっています。

コードの解説:
・voicevoxというライブラリを使用して、音声を生成するプログラムです。
・非同期処理を行うためにasyncioを使用しています。
・音声ファイルの再生にはIPython.displayを使用しています。

コードの改善点:
・不要なモジュールのインポートがありますので、削除しましょう。
・output_pathの変数は使用されておらず、削除することができます。
・音声ファイルの再生にtorchaudioとsounddeviceのどちらか一方を統一すると、コードがすっきりします。

コードの解説:
🔍 変更点がわかりやすく表示されています。
✅ レビューや改善がしやすいです。
🔄 変化を遡って確認できます。

コードの改善点:
📝 コメントの追加や見直しを行いましょう。
🔧 不要なコードや重複した部分の削除をしましょう。
🌟 パフォーマンスや可読性の向上を図りましょう。

@tuna2134 tuna2134 added the enhancement New feature or request label Aug 23, 2023
@YCHuang2112sub
Copy link
Author

output_path is used in the code.
torchaudio is for reading,
sound device is for playing.

Copy link
Member

@tuna2134 tuna2134 left a comment

Choose a reason for hiding this comment

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

LGTM!
Thank you to contribute our project.

@tuna2134 tuna2134 merged commit 091257d into voicevox-client:main Aug 23, 2023
@YCHuang2112sub
Copy link
Author

You're welcome. It's my pleasure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants