Skip to content

[Voxtral TTS] Correct decode steps param in Voxtral TTS#2524

Merged
linyueqian merged 3 commits into
vllm-project:mainfrom
y123456y78:fix-steps-param
Apr 13, 2026
Merged

[Voxtral TTS] Correct decode steps param in Voxtral TTS#2524
linyueqian merged 3 commits into
vllm-project:mainfrom
y123456y78:fix-steps-param

Conversation

@y123456y78
Copy link
Copy Markdown
Contributor

@y123456y78 y123456y78 commented Apr 6, 2026 •
edited
Loading

Purpose

  • Remove hardcoded decoding steps and added a new n_decoding_step args
  • In current voxtral tts we are actually doing n_decoding_step - 1 steps (8-1 = 7). Correct the param to call it 7 step

Test Plan

pytest -s -v   tests/model_executor/stage_input_processors/test_voxtral_tts_async_chunk.py   \
tests/model_executor/models/voxtral_tts/test_cuda_graph_acoustic_transformer.py   \
tests/model_executor/models/voxtral_tts/test_audio_tokenizer_parsing.py   \
tests/e2e/online_serving/test_voxtral_tts.py \
tests/model_executor/models/voxtral_tts/test_text_preprocess.py  \
tests/e2e/offline_inference/test_voxtral_tts.py

Test Result

image

@y123456y78
Correct decode steps param in voxtral tts
c81e206 
Signed-off-by: Chen-Yo Sun <chenyo.sun@mistral.ai>
@y123456y78 y123456y78 changed the title Correct decode steps param in voxtral tts [Voxtral TTS] Correct decode steps param in Voxtral TTS Apr 6, 2026
yenuo26 added a commit to yenuo26/vllm-omni that referenced this pull request Apr 7, 2026
@yenuo26
[Bugfix] To resolve timeout error, update nightly test commands for d…
d3814ad 
…iffusion model (vllm-project#2524)

- Exclude additional test case `test_wan_2_1_vace_expansion` from the first test command.
- Include both `test_wan22_expansion` and `test_wan2_1_vace_expansion` in the second test command.
- Update step labels for clarity.

Signed-off-by: wangyu <410167048@qq.com>
@y123456y78
Config n_decoding step
444d4dd 
Signed-off-by: Chen-Yo Sun <chenyo.sun@mistral.ai>
@y123456y78
Cleanup args loading
1a152e1 
Signed-off-by: Chen-Yo Sun <chenyo.sun@mistral.ai>
@y123456y78 y123456y78 marked this pull request as ready for review April 13, 2026 16:33
@chatgpt-codex-connector
Copy link
Copy Markdown

chatgpt-codex-connector Bot commented Apr 13, 2026

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

linyueqian
linyueqian approved these changes Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@linyueqian linyueqian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@linyueqian linyueqian added the ready label to trigger buildkite CI label Apr 13, 2026
@linyueqian linyueqian enabled auto-merge (squash) April 13, 2026 16:36
@linyueqian linyueqian merged commit c9e2e3e into vllm-project:main Apr 13, 2026
8 checks passed
Celeste-jq pushed a commit to IsleOfDawnlight/vllm-omni-voxcpm that referenced this pull request Apr 14, 2026
@y123456y78
[Voxtral TTS] Correct decode steps param in Voxtral TTS (vllm-project…
9f24901 
…#2524)

Signed-off-by: Chen-Yo Sun <chenyo.sun@mistral.ai>
lengrongfu pushed a commit to lengrongfu/vllm-omni that referenced this pull request May 1, 2026
@y123456y78 @lengrongfu
[Voxtral TTS] Correct decode steps param in Voxtral TTS (vllm-project…
0ca0dc6 
…#2524)

Signed-off-by: Chen-Yo Sun <chenyo.sun@mistral.ai>
clodaghwalsh17 pushed a commit to clodaghwalsh17/nm-vllm-omni-ent that referenced this pull request May 12, 2026
@y123456y78 @clodaghwalsh17
[Voxtral TTS] Correct decode steps param in Voxtral TTS (vllm-project…
7161696 
…#2524)

Signed-off-by: Chen-Yo Sun <chenyo.sun@mistral.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@linyueqian linyueqian linyueqian approved these changes

@hsliuustc0106 hsliuustc0106 Awaiting requested review from hsliuustc0106

Assignees

No one assigned

Labels

ready label to trigger buildkite CI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@y123456y78 @linyueqian