-
Notifications
You must be signed in to change notification settings - Fork 809
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
tensorflow-gpu==2.7.0 #790
Comments
if i update it breaks alot of stuff. |
Hello, I have the same problem and I can't find a solution. I'm running the code on Collab. Thanks in advance |
If this project wants to remain active, it will have to address the problem(s) surrounding 'pip' install. Note, running Python 3.10, and it should NOT be looking for Collecting tensorflow-gpu>=2.3.1 (from TensorFlowTTS)
Since now adays, tensorflow-gpu has been deprecated. |
|
Would it not be better to update this project? Should it not be able to support newer Python and TensorFlow? |
never tested TF-TTS with python >3.7 and TF >=2.6. |
How to run this in Colab? |
This lib should comply with tensorflow,namely
So switching this lib requirements to tensorflow instead of tensorflow-gpu should break nothing. |
This library also uses pyopenjtalk as a requirement, and this library has this build requirements:
From my understanding, these are also necessary for TensorflowTTS |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
i had same problem , solved by your answer indeed
instead of
you can find my forked version here with this and a few improvements. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
@ahmedalbahnasawi can we get it touch ? |
sure [email protected] |
All tests are passed with this diff: diff --git a/setup.py b/setup.py
index 71d6428..8a03dee 100755
--- a/setup.py
b/setup.py
@@ -22,8 22,8 @@ if LooseVersion(pip.__version__) < LooseVersion("19"):
# TODO(@dathudeptrai) update requirement if needed.
requirements = {
"install": [
- "tensorflow-gpu==2.7.0",
- "tensorflow-addons>=0.10.0",
"tensorflow>=2.7.0,<2.15.0",
"tensorflow-addons[tensorflow]>=0.10.0",
"setuptools>=38.5.1",
"huggingface_hub==0.0.8",
"librosa>=0.7.0",
diff --git a/tensorflow_tts/models/mb_melgan.py b/tensorflow_tts/models/mb_melgan.py
index 7340d84..98cb2f1 100755
--- a/tensorflow_tts/models/mb_melgan.py
b/tensorflow_tts/models/mb_melgan.py
@@ -19,7 19,7 @@
import numpy as np
import tensorflow as tf
-from scipy.signal import kaiser
from scipy.signal.windows import kaiser
from tensorflow_tts.models import BaseModel
from tensorflow_tts.models import TFMelGANGenerator Output of pip freeze:
|
why not merged? |
The text was updated successfully, but these errors were encountered: