152334H / tortoise-tts-fast

Fast TorToiSe inference (5x or your money back!)
GNU Affero General Public License v3.0
771 stars 179 forks source link

Unify inference scripts #21

Closed 152334H closed 1 year ago

152334H commented 1 year ago

There's currently:

This is too many scripts. Worse still, many of these scripts contain duplicated code to, e.g. split voices or write files.

I want:

Everything else should either be scrapped or converted to thin wrappers around the main scripts.

Ryu1845 commented 1 year ago

I actually started doing this the other day. I was planning to refactor the code a little before finishing that, but I can do it now if you want.

Ryu1845 commented 1 year ago

What do you think of using an external library like Typed Argument Parser?

152334H commented 1 year ago

I actually started doing this the other day. I was planning to refactor the code a little before finishing that, but I can do it now if you want.

Depends on how upset you are by the changes I already made earlier

What do you think of using an external library like Typed Argument Parser?

I've never tried it before, if you know it works well then go for it

Ryu1845 commented 1 year ago

Done with #27