-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
[Bug] Doesn't work with python 3.12 #3257
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. You might also look our discussion channels. |
bad bot |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. You might also look our discussion channels. |
L stale bot. Honestly the stale bot should be removed as it's causing this important issue to almost be closed for no reason. It also means I have to keep commenting for no reason.... |
This would seem like a rather urgent issue. I am not sure what the status of this project is (Coqui shut down, see https://twitter.com/_josh_meyer_/status/1742522906041635166 ) and according to Discord it's not clear who is supposed to be in charge of maintaining the repository at this point. @theoparis In the meantime do you think it might be worth throwing together a PR to address the issue? I suspect it might be very simple; see https://github.com/coqui-ai/TTS/blob/dev/setup.py#L35 Also line 120, and one should be added after 127 probably to include 3.12 for the test build, then we see where that breaks There's literally a line that throws an error during setup and exits if python version is 3.12, probably the first place to start would be commenting that out and seeing where (if anywhere) an error actually pops during execution. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. You might also look our discussion channels. |
This still seems to be an issue |
I made a WIP PR #3660 but I noticed https://github.com/coqui-ai/trainer needs to be updated as well... Not sure if I should use this fork which should fix the issue: https://github.com/eginhard/coqui-Trainer/tree/py312 |
Hey @theoparis, thanks for looking into this! If you'd like, you can make a PR to our lab's fork from where we can release a new package as well. You can change the |
Hi I added support to be able to use tts on Python 3.12 update from @theoparis to new trainer coqui-tts-trainer use this command pip install git+https://github.com/Norman742/ttspython312.git , repository https://github.com/Norman742/ttspython312.git |
New to git- just getting into things. I tried installing "pip install git+https://github.com/Norman742/ttspython312.git" with git bash on win10. got the following error: any idea why this is? |
You could try this https://stackoverflow.com/questions/75003495/error-could-not-build-wheels-for-prophet-which-is-required-to-install-pyprojec. Maybe prophet issue or need to have Microsoft Visual C++ 14.0 or greater |
Python 3.12 is now supported in our fork, available via |
Having the same issue |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. You might also look our discussion channels. |
Describe the bug
In the README, it states that "🐸TTS is tested on Ubuntu 18.04 with python >= 3.9, < 3.12.." but I don't see a tracker for 3.12 support, so opening this.
To Reproduce
Expected behavior
Working TTS (on Fedora 39).
Logs
Environment
Additional context
No response
The text was updated successfully, but these errors were encountered: