webui: add parameter groups for the ASR, VAD, diarization and alignment models - #417
Closed
CryptVenture wants to merge 1 commit into
Closed
webui: add parameter groups for the ASR, VAD, diarization and alignment models#417CryptVenture wants to merge 1 commit into
CryptVenture wants to merge 1 commit into
Conversation
…nt models Fourteen analysis families had no group in model_params.json at all, so their entire option surface was reachable only by hand-writing JSON in the fallback box -- including silero_vad's threshold and min-speech/min-silence knobs, which are the point of that model, and kroko_asr's beam search and hotword biasing. Groups added: fun_asr_nano, granite5asr, higgs_audio_stt, hviske_asr, kroko_asr, marblenet_vad, nemotron_asr, parakeet_tdt, qwen3_asr, qwen3_forced_aligner, silero_vad, sortformer_diar, vibevoice_asr, voxtral_realtime. 57 controls, each named after the key its family's session reads; the shared audio_chunk_mode and audio_chunk_duration_sec controls are read through framework/audio/chunking.cpp:167 and :190, which every one of these families calls. No existing group is touched. Validation: python3 -m json.tool webui/configs/model_params.json python3 tools/check_loader_catalog_sync.py # in sync Every control name checked against the family's own sources.
CryptVenture
force-pushed
the
pr/params-asr-groups
branch
from
September 3, 2026 01:16
a650971 to
3f9f188
Compare
Contributor
Author
|
Closing for now to stay inside the 3-concurrent-PR policy (see the discussion on #422). Nothing is wrong with the change and CI is green on it; I will reopen it as review slots free. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Split out of #374 as requested: missing parameter groups, analysis families. The TTS and music groups, the controls added to existing groups, the dead controls, the default overrides and the range fixes are separate PRs.
The problem
Fourteen ASR, VAD, diarization and forced-alignment families had no group at all in
model_params.json, so their entire option surface was reachable only by hand-writing JSON into the fallback box. That includessilero_vad'sthresholdand min-speech/min-silence knobs — which are the point of that model — andkroko_asr's beam search and hotword biasing.The change
Groups added:
fun_asr_nano,granite5asr,higgs_audio_stt,hviske_asr,kroko_asr,marblenet_vad,nemotron_asr,parakeet_tdt,qwen3_asr,qwen3_forced_aligner,silero_vad,sortformer_diar,vibevoice_asr,voxtral_realtime— 57 controls in total.Every control is named after the key its family's session actually reads. The shared
audio_chunk_modeandaudio_chunk_duration_seccontrols are read throughsrc/framework/audio/chunking.cpp:167and:190, which each of these families calls; the rest resolve inside the family's own sources.No existing group is touched, so nothing that worked before changes.
Validation
Every one of the 57 control names was checked against its family's sources — either a literal
"<name>"in the family directory, or the shared chunking helper above.Scope
One data file, 85 added lines, no deletions. No code change. The generated bundle is deliberately excluded —
catalog.tsinlines this file at frontend build time and the bundle is not byte-reproducible, so regenerating it in each PR of this split would make the PRs conflict; happy to send one bundle-regeneration PR once the series lands.