Skip to content
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

Added serverless API #282

Closed
wants to merge 7 commits into from
Closed

Added serverless API #282

wants to merge 7 commits into from

Conversation

milo157
Copy link

@milo157 milo157 commented Oct 22, 2024

Added the ability to run this as a serverless API on Cerebrium

@rany2
Copy link
Owner

rany2 commented Oct 22, 2024

Thanks for your PR! The new docs for the serverless API are well-written too. I'll merge it in a bit and might try to add support for other serverless platforms.

@milo157
Copy link
Author

milo157 commented Oct 24, 2024

@rany2 I think I fixed the formatting issues

@rany2
Copy link
Owner

rany2 commented Nov 8, 2024

I won't merge it. I don't really want this to be used outside of personal use and I think deploying to a serverless API encourages this use-case. Microsoft is already cracking down on this library in Mainland China as it's being used heavily for commercial reasons.

Sorry about this but I've already had to deal with adding a new DRM they implemented to make it not work in Mainland China: #303

It's really odd that Chinese companies are using it commercially as the Azure API isn't really expensive in the first place and I've never encouraged this use-case when people asked me if it's OK to, but whatever.

I initially used it to generate e-books and later to generate short <10second MP3s to use in Home Assistant, later it became the only way to use this service on Linux because Microsoft dropped support for the TTS on Linux for some reason :')

@rany2 rany2 closed this Nov 8, 2024
@zhongtanru
Copy link

Edge-tts is good for teaching and learning English. We English teachers need it.

@rany2
Copy link
Owner

rany2 commented Nov 8, 2024

@zhongtanru It should work again with edge-tts version 6.1.16 :)

@zhongtanru
Copy link

zhongtanru commented Nov 8, 2024 via email

@zhongtanru
Copy link

zhongtanru commented Nov 11, 2024 via email

@rany2
Copy link
Owner

rany2 commented Nov 11, 2024

@zhongtanru I tried implementing a fix that might help related to clock skew. You can try it with edge-tts version 6.1.18.

@zhongtanru
Copy link

zhongtanru commented Nov 11, 2024 via email

@zhongtanru
Copy link

zhongtanru commented Nov 22, 2024 via email

@zhongtanru
Copy link

zhongtanru commented Nov 22, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants