Repository navigation
Commit 7965efd
Fix the Windows Voxtral Realtime CI test (#24289)
The Windows Voxtral Realtime test could never pass, for two reasons.
Both came in with #17993, which added `-Mode` to
`test_model_e2e_windows.ps1` with streaming as the default and switched
the expected word to "Quilter", without updating the workflow that calls
the script.
- The workflow exports the model in offline mode but called the test
without `-Mode`, so the script ran the runner with `--streaming`. The
runner refuses `--streaming` for a model exported without it and aborts.
Release builds compile out the log, so the job only showed exit code
0xC0000409. The mode now lives in one place: a `mode` key on the Voxtral
Realtime row of both matrices, passed by the export and the test step
alike. Other rows have no key and pass an empty mode, which both scripts
treat as before.
- The test looked for "Quilter" in a transcript of a poem recording that
does not contain that word. "Quilter" comes from the LibriSpeech clip
the Linux test uses. The test now uses the LibriSpeech clip the Windows
Parakeet test already downloads (saved as `test_audio.wav` instead of
`poem.wav`) and looks for "Phoebe", as the MLX Voxtral Realtime test
does with the same clip.
A second commit adds `export_model_artifact.sh` and
`test_model_e2e_windows.ps1` to the workflow's paths filter and to the
export and test job conditions, as cuda.yml does for its own scripts.
Before, a change that only touched those scripts ran no Windows model
job.
Testing: on Linux CUDA, a Release runner with logging off reproduces the
Windows abort with the old flags, and with the new ones transcribes the
clip and prints "Phoebe". This branch lives in the main repository so
the Windows CUDA jobs run on this PR; on the previous head all six
Windows model tests passed, including Voxtral Realtime with "Phoebe".
Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>1 parent b80295b commit 7965efd
2 files changed
Lines changed: 13 additions & 9 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
89 | 89 | | |
90 | 90 | | |
91 | 91 | | |
92 | | - | |
| 92 | + | |
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
96 | | - | |
97 | | - | |
| 96 | + | |
| 97 | + | |
98 | 98 | | |
99 | 99 | | |
100 | 100 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| 20 | + | |
| 21 | + | |
20 | 22 | | |
21 | 23 | | |
22 | 24 | | |
| |||
61 | 63 | | |
62 | 64 | | |
63 | 65 | | |
| 66 | + | |
| 67 | + | |
64 | 68 | | |
65 | 69 | | |
66 | 70 | | |
| |||
92 | 96 | | |
93 | 97 | | |
94 | 98 | | |
| 99 | + | |
95 | 100 | | |
96 | 101 | | |
97 | 102 | | |
| |||
147 | 152 | | |
148 | 153 | | |
149 | 154 | | |
150 | | - | |
151 | | - | |
152 | | - | |
153 | | - | |
154 | | - | |
| 155 | + | |
155 | 156 | | |
156 | 157 | | |
157 | 158 | | |
| |||
168 | 169 | | |
169 | 170 | | |
170 | 171 | | |
| 172 | + | |
| 173 | + | |
171 | 174 | | |
172 | 175 | | |
173 | 176 | | |
| |||
195 | 198 | | |
196 | 199 | | |
197 | 200 | | |
| 201 | + | |
198 | 202 | | |
199 | 203 | | |
200 | 204 | | |
| |||
228 | 232 | | |
229 | 233 | | |
230 | 234 | | |
231 | | - | |
| 235 | + | |
232 | 236 | | |
233 | 237 | | |
234 | 238 | | |
| |||
0 commit comments