Skip to content

Fix gradio passing overrides to tts.py#13

Merged
randombk merged 1 commit intorandombk:masterfrom
odie5533:fix-arg-overrides
Aug 30, 2025
Merged

Fix gradio passing overrides to tts.py#13
randombk merged 1 commit intorandombk:masterfrom
odie5533:fix-arg-overrides

Conversation

@odie5533
Copy link
Copy Markdown
Contributor

@odie5533 odie5533 commented Aug 29, 2025

@randombk This vLLM version is incredible! I'm getting like 16-18x speedup in batch mode! Thank you so much for making it! I have no idea how you did it, but it's amazing!

  • Fix gradio passing overrides to tts.py
  • Add diffusion steps to gradio with default 10

I found the audio quality of the vLLM to be appreciably worse than the original and went hunting for the reason. I saw the diffusion steps were set down and set them up and that fixed it! So I'd recommend defaulting to 10. vLLM is still loads faster than the original with 10 steps, so I think letting people manually set down the quality for even more speed is better.

Also fixes errors like

  File "/home/david/chatterbox-vllm/gradio_tts_app.py", line 29, in load_model
    global_model = ChatterboxTTS.from_pretrained(
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/david/chatterbox-vllm/src/chatterbox_vllm/tts.py", line 165, in from_pretrained
    return cls.from_local(Path(local_path).parent, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/david/chatterbox-vllm/src/chatterbox_vllm/tts.py", line 123, in from_local
    t3 = LLM(
         ^^^^
TypeError: vllm.entrypoints.llm.LLM() got multiple values for keyword argument 'gpu_memory_utilization'

* Add diffusion steps to gradio with default 10
@sipvoip
Copy link
Copy Markdown

sipvoip commented Aug 29, 2025

Thanks, this is great!

@randombk
Copy link
Copy Markdown
Owner

LGTM, and thanks for the feedback on the step count - I'll change it back to the default 10.

The gpu_memory_utilization issue was me forgetting the update Gradio when I changed the API - I'll send out a proper fix for that in a bit.

@randombk randombk merged commit 8dbf8c2 into randombk:master Aug 30, 2025
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