Skip to content

Added 60dB integration#26

Closed
manishEMS47 wants to merge 1 commit into
digitalsamba:mainfrom
manishEMS47:main
Closed

Added 60dB integration#26
manishEMS47 wants to merge 1 commit into
digitalsamba:mainfrom
manishEMS47:main

Conversation

@manishEMS47

Copy link
Copy Markdown

No description provided.

@ConalMullan

Copy link
Copy Markdown
Collaborator

Thanks for this, @manishEMS47 — really solid integration, and the Hindi/Indian-voice angle fills a genuine gap for us. I tested it against a live 60db key and found the only real issue: the code follows 60db's documented /tts-synthesize contract (single JSON audio_base64), but the live endpoint actually streams NDJSON of raw 48 kHz PCM — so the default path errored with "Invalid JSON response."

I've built on your commit (preserved with your authorship) and fixed the response handling to work against the live API, with end-to-end verification (EN + Hindi → valid MP3, voiceover.py --provider 60db per-scene). That's in #27, which credits this PR and closes it on merge. Thanks again! 🙌

@ConalMullan

Copy link
Copy Markdown
Collaborator

Closing in favour of #27, which builds directly on this work (your original commit is preserved with your authorship) and adds the live-API fix needed to make 60db work end-to-end — the production /tts-synthesize endpoint streams NDJSON of raw 48 kHz PCM rather than the documented single-JSON audio_base64 mp3, so the original path errored. Thanks again for kicking this off, @manishEMS47 — the Hindi/Indic voice support fills a real gap. Follow along on #27. 🙌

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.

2 participants