Better error handling of invalid parameters.

[jmtatsch]

Here, I had by chance the non existing voice 'alls'

server-1  |   File "/app/speech.py", line 141, in generate_speech
server-1  |     piper_model, speaker = map_voice_to_speaker(voice, 'tts-1')
server-1  |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
server-1  |   File "/app/speech.py", line 90, in map_voice_to_speaker
server-1  |     return (voice_map[model][voice]['model'], voice_map[model][voice]['speaker'])
server-1  |             ~~~~~~~~~~~~~~~~^^^^^^^
server-1  | KeyError: 'alls'

[jmtatsch]

It doesn't really die, this was premature...

[matatonic]

it should error back to the user instead of this behavior. I'll rename & reopen the bug for tracking.

Thanks for the report.

[JohnSmithToYou]

Yes, especially with the /v1/audio/speech endpoint. If you pass in a model other than "tts-1-hd" or "tts-1" you get:

server-1  |   File "/app/speech.py", line 193, in generate_speech
server-1  |     ffmpeg_args.extend(["-"])

[matatonic]

Good point, thanks for the feedback!