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

Update ASpeech.py #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Yash-2707
Copy link

@Yash-2707 Yash-2707 commented Oct 28, 2024

I changed the function names to follow PEP 8 conventions (lowercase with underscores)
I moved the strip function inside the ASpeech class to make it a method
I added a print statement to the play function to provide feedback.
I added a try-except block to the process_text function to handle potential errors.
I used daemon threads to allow the main thread to exit even if the threads are still running.
I organized the code into logical sections to improve readability.

 I changed the function names to follow PEP 8 conventions (lowercase with underscores)
 I moved the strip function inside the ASpeech class to make it a method
 I added a print statement to the play function to provide feedback.
I added a try-except block to the process_text function to handle potential errors.
I used daemon threads to allow the main thread to exit even if the threads are still running.
I organized the code into logical sections to improve readability.
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.

1 participant