Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Upgrade python versions to allow 3.11 installation and fix current pip installation failure#27

Open
jmfrank63 wants to merge 10 commits into
rhasspy:masterfrom
jmfrank63:fix-python-install
Open

Upgrade python versions to allow 3.11 installation and fix current pip installation failure#27
jmfrank63 wants to merge 10 commits into
rhasspy:masterfrom
jmfrank63:fix-python-install

Conversation

@jmfrank63

Copy link
Copy Markdown

Currently tested on a Mac M1, this alongside the piper PR should fix
pip install piper-tts, which already created three issues.

Upgraded several python versions.
Windows and Linux, not tested yet.

@jmfrank63

Copy link
Copy Markdown
Author

This is now at a point, where some conversation should take place. Currently, MacOS and Windows are working and roughly tested. Linux will follow, but I do not expect the same chore as with Windows.
Both this and the piper are modified. If you want to run this, set an environment variable PIPER_PHONEMIZE_REPO and point it to your fork, so piper will load it.

@rmcpantoja

rmcpantoja commented Feb 25, 2024

Copy link
Copy Markdown

This is now at a point, where some conversation should take place. Currently, MacOS and Windows are working and roughly tested. Linux will follow, but I do not expect the same chore as with Windows. Both this and the piper are modified. If you want to run this, set an environment variable PIPER_PHONEMIZE_REPO and point it to your fork, so piper will load it.

Hi @jmfrank63,
How did you manage to do a wheel on Windows?

I tried through GitHub actions but got errors:
https://github.com/rmcpantoja/piper-phonemize/blob/master/.github/workflows/main.yml

@jmfrank63

Copy link
Copy Markdown
Author

I don't know exactly what kind of errors you are experiencing, but it might be the Unicode that is not understood by the Microsoft C-compiler. I changed the utf-8 characters to the hex representation and added conditional compilation for windows. But I must have made a mistake because it broke other languages except English, which now produce pure nonsense. Haven't had the time to look into this yet.

@rmcpantoja

Copy link
Copy Markdown

I don't know exactly what kind of errors you are experiencing, but it might be the Unicode that is not understood by the Microsoft C-compiler. I changed the utf-8 characters to the hex representation and added conditional compilation for windows. But I must have made a mistake because it broke other languages except English, which now produce pure nonsense. Haven't had the time to look into this yet.

You can see the full log here. It says it can't find espeak-ng/include, maibe I'm doing pip wheel in an wrong path...

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants