Hi @begeekmyfriend! It looks like your repo requires tensorflow only for the HParams class. You might be interested in this very simple replacement. This lets you remove the tensorflow requirement.
My version of the HParams class supports overrides with hparams.parse() but saves a lot of code by not performing validation to ensure that the overridden hparams have the correct datatype. This is code that I developed while removing tensorflow from another repo that is based on Rayhane's tacotron-2: https://github.com/CorentinJ/Real-Time-Voice-Cloning/pull/472
I'll note that I am unable to test the changes because I am new to your repo and have not set it up yet. However, it is a small change and you should be able to validate it easily.
begeekmyfriend
commented
4 years ago
Hi @begeekmyfriend! It looks like your repo requires tensorflow only for the HParams class. You might be interested in this very simple replacement. This lets you remove the tensorflow requirement.
My version of the HParams class supports overrides with
hparams.parse()but saves a lot of code by not performing validation to ensure that the overridden hparams have the correct datatype. This is code that I developed while removing tensorflow from another repo that is based on Rayhane's tacotron-2: https://github.com/CorentinJ/Real-Time-Voice-Cloning/pull/472I'll note that I am unable to test the changes because I am new to your repo and have not set it up yet. However, it is a small change and you should be able to validate it easily.