Ml model config registry dispatch
Skill kjuhwa/skills-hub/skills/architecture/ml-model-config-registry-dispatch
Centralize per-engine ML model configuration in a declarative registry so route, service, and backend layers stay free of if/elif chains.From its SKILL.md
npx -y skills add kjuhwa/skills-hub --skill ml-model-config-registry-dispatchAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
3.2 KB, 627 tokens by cl100k_base, as published. Nobody here has run it
ML model config registry dispatch
When to use
Your app supports multiple ML engines (TTS/STT/image) with per-engine quirks: different HF repos, size variants, languages, feature flags (instruct support, hallucination trim, etc.). Left unchecked, the same if/elif chain spreads across routes, services, UI selectors, and download screens.
Steps
- Define a
@dataclass ModelConfigwith fields:model_name(stable id),display_name,engine,hf_repo_id,model_size(default"default"),size_mb, plus any boolean feature flags relevant to your domain (needs_trim,supports_instruct, etc.) and alanguages: list[str]. - Write small generator functions per engine family (
_get_qwen_model_configs(),_get_non_qwen_tts_configs(), etc.) that returnlist[ModelConfig]. Centralize platform branching (CUDA vs MLX repos) inside the generator so callers never see it. - Expose aggregate accessors:
get_all_model_configs(),get_tts_model_configs(),get_model_config(model_name). Downstream code uses these — never hardcodes an engine name. - Route lookups through the registry. Replace
if engine == "qwen": ... elif engine == "chatterbox": ...withengine_needs_trim(engine),engine_has_model_sizes(engine),get_tts_backend_for_engine(engine), etc. Each is a one-liner over the registry. - Keep a thread-safe factory (
dict + Lock) that lazily instantiates backend classes on first access and caches them. Double-check-locked to avoid duplicate instantiation under load. - Unify load/unload/check through config-driven helpers (
unload_model_by_config(cfg),check_model_loaded(cfg),get_model_load_func(cfg)) so the HTTP routes become single-call functions with no engine branching.
Counter / Caveats
- Two engines cannot share a
model_name. Treatmodel_nameas the stable public id used by HTTP payloads and DB rows, and never let the display name leak into persistence. - When a new feature applies to only one engine (e.g.
supports_instructfor Qwen CustomVoice), add the flag toModelConfigand set it per-engine rather than branching by engine name downstream. - Do not put runtime state (loaded model, weights) on the config. Configs are declarative; the backend instance holds runtime state.
- Per-engine backend classes still expose their own tiny dispatch for load signatures (
load_model(model_size)vsload_model()) — keep that dispatch in one place (load_engine_model) rather than rediscovering it per caller.
Source references: backend/backends/__init__.py (the ModelConfig, config generators, lookup helpers, get_tts_backend_for_engine, load_engine_model, check_model_loaded, unload_model_by_config).
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.